[FFmpeg-devel] [PATCH] flv duration
Sun Apr 4 03:41:24 CEST 2010
Howard Chu wrote:
> Michael Niedermayer wrote:
>> On Thu, Apr 01, 2010 at 02:28:11AM -0700, Howard Chu wrote:
>>> When playing an FLV, you always see this message:
>>> [flv @ 0x120cea0]Estimating duration from bitrate, this may be inaccurate
>>> even if the FLV metadata provides the duration. I'd been ignoring this for
>>> the most part, but after someone else mentioned it on irc I decided to take
>>> a look. The FLV demux sets the AVFormatContext->duration based on the
>>> metadata, but the av_has_duration() function wants to see it set in one of
>>> the AVStreams. It looks to me like it does no harm one way or the other,
>>> but I wrote this patch just to shut up that diagnostic.
>> you dont need to set s->duration
> OK. Works fine, make test passes.
Spoke too soon. Not setting s->duration breaks XBMC.
-- Howard Chu
CTO, Symas Corp. http://www.symas.com
Director, Highland Sun http://highlandsun.com/hyc/
Chief Architect, OpenLDAP http://www.openldap.org/project/
More information about the ffmpeg-devel