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

Mathieu Duponchelle mathieu at centricular.com
Fri Apr 19 15:46:22 EEST 2019


Hello ? :)

On 4/10/19 1:26 PM, Mathieu Duponchelle wrote:
> 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".
> _______________________________________________
> 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