[FFmpeg-devel] [PATCh] aac parser: Don't write channels, sample rate, and frame size each frame

[Alex Converse]

On Mon, Mar 1, 2010 at 8:31 PM, Alex Converse <alex.converse at gmail.com> wrote:
>
>
> On Mon, Mar 1, 2010 at 8:23 PM, Michael Niedermayer <michaelni at gmx.at> wrote:
>>
>> On Mon, Mar 01, 2010 at 08:16:48PM -0500, Alex Converse wrote:
>> > They are part of the invariant header and shouldn't change each frame
>> > anyway. This prevents them from overwriting information provided by
>> > backwards compatible extensions.
>>
>> what you attached does not match the description
>>
>> besides "shouldn't change" gives me a strange feeling ...
>> if we can support it changing it would be a nice "wish/feature request"
>
> According to the AAC specification they changing these vales from frame to frame is forbidden. They are part of the fixed header.
>

Oh also to support those sorts of illegal files we'd need an overhaul
to the AAC output configuration locking system.

And until we actually support them rejecting them has done a lot to
improve the stability (in a software sense, not a numerical sense) of
the decoder.