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

Martin Storsjö martin
Mon Sep 7 08:47:09 CEST 2009


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?

// Martin



More information about the ffmpeg-devel mailing list