[FFmpeg-devel] [PATCH] avcodec/aac_adtstoasc: recover original extradata if the stream is already ASC

James Almer jamrial at gmail.com
Fri Nov 25 02:08:59 EET 2016


On 11/24/2016 8:13 PM, Hendrik Leppkes wrote:
> On Tue, Nov 22, 2016 at 3:13 AM, James Almer <jamrial at gmail.com> wrote:
>> Fixes ticket #5973
>>
>> Signed-off-by: James Almer <jamrial at gmail.com>
>> ---
>> Maybe init() should propagate the extradata down the filter chain instead
>> of this, since if an aac stream has extradata then it means that it's an
>> ASC stream. Neither ADTS or LATM use extradata.
>>
>> What's preferred?
>>
> 
> Forwarding the extradata right away in init is probably better. As you
> say any of the other alternatives don't use any.
> Could try to parse it to make sure its valid, and not forwarding
> garbage, if you really care.

Sure, will send a new patch to do that. Thanks.

> 
> - Hendrik
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> 



More information about the ffmpeg-devel mailing list