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

Michael Niedermayer michaelni at gmx.at
Mon Jun 18 00:35:52 CEST 2012

On Sun, Jun 17, 2012 at 11:27:25PM +0200, Nicolas George wrote:
> 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?

Its not hard to add such field to nut if it is neccessary.
maybe this should be discussed on nut-devel, and CC ffmpeg-dev


Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

There seems to be only one solution to NIH syndrom, ... a shooting squad
-------------- 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/20120618/a0daeb53/attachment.asc>

More information about the ffmpeg-devel mailing list