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

Hendrik Leppkes h.leppkes at gmail.com
Sun Jun 17 21:43:55 CEST 2012


On Sun, Jun 17, 2012 at 8:55 PM, Philip Langdale <philipl at overt.org> wrote:
>
> 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.
>
>
AFAIK, convergence_duration is used because its int64 and duration only int.
On some matroska time bases the duration of a subtitle packet (multiple
seconds) can manage to overflow the 32bit duration.

At least thats the only useful reason i remember.


More information about the ffmpeg-devel mailing list