[FFmpeg-devel] [PATCH] latmenc: Check for LOAS sync word

Paul B Mahol onemda at gmail.com
Wed Jan 16 21:49:31 CET 2013


On 1/16/13, James Almer <jamrial at gmail.com> wrote:
> On 16/01/13 5:17 PM, Michael Niedermayer wrote:
>> If extradata is set then ADTS would be muxed into LATM creating a
>> invalid output
>> also the code is indented in a way that does not match the actual
>> relation of the statements
>
> Encoders should not send ADTS packets with extradata. None currently does
> either.

It would be good if it is checked if possible anyway.

> And i left indentation like that on purpose as other people asked me to
> do, to make the review easier (Leaving that for a cosmetic patch).
>
> Do you want me to keep the ADTS check untouched and outside the extradata
> check then?
>
> Regards.
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>


More information about the ffmpeg-devel mailing list