On Thu, Jan 5, 2012 at 4:33 PM, Alex Converse <[email protected]> wrote:
> This keeps internal state variable accurate for LFEs and any other
> elements which lack SBR data payloads.
> ---
> libavcodec/aacsbr.c | 2 ++
> 1 files changed, 2 insertions(+), 0 deletions(-)
>
> diff --git a/libavcodec/aacsbr.c b/libavcodec/aacsbr.c
> index 81b0b4c..d38debc 100644
> --- a/libavcodec/aacsbr.c
> +++ b/libavcodec/aacsbr.c
> @@ -129,6 +129,8 @@ av_cold void ff_aac_sbr_init(void)
> av_cold void ff_aac_sbr_ctx_init(AACContext *ac, SpectralBandReplication
> *sbr)
> {
> float mdct_scale;
> + sbr->sample_rate = 2 * ac->m4ac.sample_rate; //TODO use the nominal
> sample rate for arbitrary sample rate support
> + ac->m4ac.ext_sample_rate = 2 * ac->m4ac.sample_rate;
This breaks HE-AAC in ADTS {fate-aac-ct-sbr_i-ps_i.aac} because
m4ac.sample_rate is set by compatibility for
headerless-AAC-sans-extradata.
> sbr->kx[0] = sbr->kx[1] = 32; //Typo in spec, kx' inits to 32
> sbr->data[0].e_a[1] = sbr->data[1].e_a[1] = -1;
> sbr->data[0].synthesis_filterbank_samples_offset = SBR_SYNTHESIS_BUF_SIZE
> - (1280 - 128);
> --
> 1.7.3.1
>
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
