[FFmpeg-devel] [PATCH] matroskadec: set duration also for subtitles.

Nicolas George nicolas.george at normalesup.org
Sun Jun 17 21:00:57 CEST 2012

Le decadi 30 prairial, an CCXX, Philip Langdale a écrit :
> I had a slightly different patch in my queue. Is the setting of
> convergence_duration logical in any way? Shouldn't we just set the
> normal duration unconditionally and not bother with convergence_duration
> for subtitles? TEXT is the only one where we do it, and it seems to make
> no sense whatsoever.

I agree that convergence_duration makes no sense, even stretching the doc a
lot. But it has been like that for a long time, removing it now would break
programs that rely on it. And since it was the only way to get that
information (unlike ASS, where the end timestamp is resynthetized in the
packet data), we can not blame these programs.

So I think we are stuck with setting convergence_duration for TEXT, at least
for the time being. I can add a comment about that if people want it.


  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120617/8899483e/attachment.asc>

More information about the ffmpeg-devel mailing list