[FFmpeg-devel] [PATCH] matroskadec, matroskadec, srtenc: Read/Write duration for subtitles. (v2)

[Clément Bœsch]

On Sun, Aug 05, 2012 at 06:21:25PM +0200, Hendrik Leppkes wrote:
> On Sun, Aug 5, 2012 at 6:17 PM, Philip Langdale <philipl at overt.org> wrote:
> 
> > > There are files with such a timebase. It may not be common, but they
> > > do exist and they are valid according to the MKV spec.
> > > So, yes, it is needed for such files, until such a day when the real
> > > duration is converted to int64 (if ever)
> >
> > Ugh. This is such a mess. If we really can't move mkv to use duration,
> > then we've got to go in the other direction.
> >
> > Maybe:
> >
> > #define SUBTITLE_DURATION convergence_duration
> >
> > and use pkg->SUBTITLE_DURATION for now and switch the
> > define when we can finally break compatibility.
> >
> >
> As an alternative, could detect such timebases and scale them by factor
> 1000 to microseconds (1/1000000), with a slight loss of precision, of
> course.

I second that. It's likely that precision wasn't here at muxing time
anyway.

-- 
Clément B.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 490 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120805/f891fe90/attachment.asc>