[FFmpeg-devel] [PATCH] better detection of available streams for flv

Michael Niedermayer michaelni
Sun Apr 13 19:57:44 CEST 2008


On Sun, Apr 13, 2008 at 06:06:00PM +0200, Reimar D?ffinger wrote:
> Hello,
> with e.g. http://eurovision.tv/upload/video/2008/WELCOMETOBELGRADE.flv
> the flv demuxer thinks that there is no video stream.
> Attached code changes the code to create a audio/video stream if the
> metadata-parsing code finds a key starting with "audio" or "video" and
> there is not yet such a stream.
> Obviously, there are many other ways to do this, and if such a change is
> done the hack in flv_read_header should probably be removed.
> Instead of checking for any key starting with audio/video this could of
> course be restricted to those we actually handle, namely audiocodecid,
> videocodecid, audiosamplesize, audiosamplerate and stereo.

We support flv with invalid headers since r10887. Which does not depend on
metadata. Why does this not suffice?

Also width/height for example have no video prefix.

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I count him braver who overcomes his desires than him who conquers his
enemies for the hardest victory is over self. -- 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/20080413/9f22d7d0/attachment.pgp>



More information about the ffmpeg-devel mailing list