[FFmpeg-devel] [PATCH 3/3] ffmpeg: ensure that -fix_sub_duration doesnt create subtitles with zero duration
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:
>> 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
More information about the ffmpeg-devel