[FFmpeg-devel] [PATCH] mpeg12enc: Use all Closed Captions side data

Mathieu Duponchelle mathieu at centricular.com
Wed Apr 10 14:26:22 EEST 2019


No problem, note that the added indentation is because a loop
was added :)

On 4/10/19 1:14 AM, Carl Eugen Hoyos wrote:
> 2019-03-23 0:22 GMT+01:00, Mathieu Duponchelle <mathieu at centricular.com>:
>> It is perfectly valid to have multiple CC Picture User Data
>> for the same frame. Instead of using the first side_data
>> potentially present with the A53_CC type, iterate over all
>> side_data.
> Please separate the functional changes in your patch from
> the cosmetics, ie do not re-indent in the same patch to make
> reviews easier.
>
> Sorry for the late reply, Carl Eugen
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".


More information about the ffmpeg-devel mailing list