[FFmpeg-devel] [PATCH] Use all 32 bits of the timestamp when calculating flv duration
Thu Sep 3 15:48:44 CEST 2009
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.
> also the code could maybe be factored out, we have that 24+8 reading in there
That might be a good idea...
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 646 bytes
More information about the ffmpeg-devel