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

Nicolas George 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
   convergence_duration;
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?

Regards,

-- 
  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/3e500c58/attachment.asc>


More information about the ffmpeg-devel mailing list