[FFmpeg-devel] [PATCH] shortcut av_find_stream_info for flvdec

[Michael Niedermayer]

On Wed, Apr 21, 2010 at 11:41:06PM +0200, Michael Niedermayer wrote:
> On Wed, Apr 21, 2010 at 01:32:07PM -0700, Howard Chu wrote:
> > Michael Niedermayer wrote:
> >> On Wed, Apr 21, 2010 at 12:51:37AM -0700, Howard Chu wrote:
> >>> This patch pulls more codec parameters out of the FLV metadata. It cuts 
> >>> the
> >>> number of packets needed in av_find_stream_info() from ~60 down to just 
> >>> 2.
> >>
> >> this patch reverts bugfixes, and introduces more bugs then there where
> >> see r19362 and issue977
> >>
> >> PS: maybe we should add te file from issue977 to fate ...
> >
> > Bummer. Thanks for the references.
> >
> > Can someone comment on why tb_unreliable() says the H264 codec timebase is 
> > unreliable?
> 
> i would guess telecine but there might have been other things i am just
> missing atm
> 
> 
> > If it has a variable frame rate, how does checking the first 20 
> > frames tell you any more about it than just the first 2 frames?
> 
> checking more frames will give a more accurate estimate, its still an
> estimate though

btw some means to allow a user app to specify which things it needs and which
not (like the r_frame_rate) would be welcome

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Many that live deserve death. And some that die deserve life. Can you give
it to them? Then do not be too eager to deal out death in judgement. For
even the very wise cannot see all ends. -- Gandalf
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100421/8486edc2/attachment.pgp>