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

[Michael Niedermayer]

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>