[FFmpeg-devel] [PATCH] matroskadec: set duration also for subtitles.
nicolas.george at normalesup.org
Sun Jun 17 23:27:25 CEST 2012
Le decadi 30 prairial, an CCXX, Philip Langdale a écrit :
> Ugh. This complicates things a fair bit. Should we just standardize
> on convergence_duration for all subtitle AVPackets then? It'll suck
> to have to check both fields everywhere.
That is true, but the other solutions have drawbacks too.
1. allow demuxers to store either duration or convergence_duration;
2. check that all demuxers make a special case for subtitles and store
3. in av_read_frame, copy duration into convergence_duration if necessary.
Solution 3 seems the less complex, but it is not very elegant.
By the way, I just noticed that NUT does not store any duration at all. What
should we do about it?
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 198 bytes
Desc: Digital signature
More information about the ffmpeg-devel