[FFmpeg-devel] [PATCH] Use all 32 bits of the timestamp when calculating flv duration

Diego Biurrun diego
Mon Sep 7 12:50:04 CEST 2009


On Mon, Sep 07, 2009 at 09:47:09AM +0300, Martin Storsj? wrote:
> On Thu, 3 Sep 2009, Martin Storsj? wrote:
> 
> >On Thu, 3 Sep 2009, Michael Niedermayer wrote:
> >
> >>On Thu, Sep 03, 2009 at 04:18:50PM +0300, Martin Storsj? wrote:
> >>>
> >>>$subj. At the moment, the duration is mainly set from the metadata 
> >>>packet,
> >>>but if that isn't available, the fallback by checking the timestamp of 
> >>>the
> >>>last packet really should read all 32 bits of the timestamp, not just the
> >>>low 24 bits as currently.
> >>
> >>ok if its confirmed that this does fix a bug
> >
> >Yes, I tested it that it fixes a bug, otherwise the duration is wrong for 
> >all files over 4,6 hours.
> >
> >Attached a version that adds proper spacing around the equals sign, so 
> >that we clean up whitespace for lines that we're touching anyway.
> 
> Ping, anyone care to apply this?

Applied.

Diego



More information about the ffmpeg-devel mailing list