[FFmpeg-devel] [PATCH 3/3] ffmpeg: ensure that -fix_sub_duration doesnt create subtitles with zero duration

Marton Balint cus at passwd.hu
Thu Nov 28 21:40:58 CET 2013


On Mon, 25 Nov 2013, Nicolas George wrote:

> Le tridi 3 frimaire, an CCXXII, Marton Balint a écrit :
>> Yes, it is on purpose. As w4m pointed out in response to my earlier patch:
>>
>> https://lists.ffmpeg.org/pipermail/ffmpeg-devel/2013-November/thread.html#151037
>>
>> 0 duration can also mean unknown (infinite) duration depending on
>> the codecs used, so it would be wrong to drop them all.
>>
>> However, when fix_sub_duration is used, and the duration fixing code
>> is generating 0 duration, that is definitely zero, and not undefined
>> or infinite, so it is safe to drop it.
>
> Thanks for explaining. I suppose this patch is ok, then.
>

I have updated the patch descriptions based on the questions asked, and 
pushed them to my stable branch.

Michael, could you please merge them? Nicolas sort of reviewed all three:

3e396ca ffmpeg: do not pass possibly undefined subtitles to sub2video_update
4d6f2ff ffmpeg: free empty subtitles as well in transcode_subtitles
31bb172 ffmpeg: ensure that -fix_sub_duration doesnt create subtitles with zero duration

Thanks,
Marton


More information about the ffmpeg-devel mailing list