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

[Michael Niedermayer]

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 was thinking of this as well and think this would be the better
approuch over a mix of 2 fields

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I have often repented speaking, but never of holding my tongue.
-- Xenocrates
-------------- 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/20120805/3f949979/attachment.asc>