[FFmpeg-devel] [PATCH 24/28] changed: only require first?packet?to be known for all audio and video streams

Michael Niedermayer michaelni
Fri Jul 2 16:17:48 CEST 2010

On Fri, Jul 02, 2010 at 10:10:30AM +0000, Joakim Plate wrote:
> Michael Niedermayer <michaelni <at> gmx.at> writes:
> > 
> > iam curious what this is supposed to improve
> > though i dont like this change, it looks ugly and hackish
> > 
> > [...]
> For subtitle stream there may not be any data during even in a huge probe size. 
> No point in waiting for the subtitle data or data streams to show up in 
> av_find_stream_info

thats an argument for ==SUBTITLE || ==DATA
but not for !=VIDEO && !=AUDIO
with the first variant and a comment explaining the reasoning behind
i guess the patch would be ok 

Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The educated differ from the uneducated as much as the living from the
dead. -- Aristotle 
-------------- 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/20100702/f98f4678/attachment.pgp>

More information about the ffmpeg-devel mailing list