Re: [FFmpeg-devel] [PATCH 5/8] Support encoding of Active Format Description (AFD) in libx264

2018-01-05 Thread Devin Heitmueller
Hi Aaron, >> +ret = ff_alloc_afd_sei(frame, 0, _data, _size); >> +if (ret < 0) { >> +for (i = 0; i < num_payloads; i++) >> +av_free(x4->pic.extra_sei.payloads[i].payload); >> +av_free(x4->pic.extra_sei.payloads); > >

Re: [FFmpeg-devel] [PATCH 5/8] Support encoding of Active Format Description (AFD) in libx264

2017-12-29 Thread Aaron Levinson
On 12/29/2017 10:12 AM, Devin Heitmueller wrote: If AFD side data is present, include it in an H.264 SEI payload when encoding with libx264. This is done in the same manner that we currently handle A53 closed captions (where the business logic for constructing the SEI is in libavcodec/utils.c),

[FFmpeg-devel] [PATCH 5/8] Support encoding of Active Format Description (AFD) in libx264

2017-12-29 Thread Devin Heitmueller
If AFD side data is present, include it in an H.264 SEI payload when encoding with libx264. This is done in the same manner that we currently handle A53 closed captions (where the business logic for constructing the SEI is in libavcodec/utils.c), so it should be portable to the other encoder