[FFmpeg-devel] [PATCH] matroskadec: set duration also for subtitles.
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
> 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
Size: 198 bytes
Desc: Digital signature
More information about the ffmpeg-devel