[FFmpeg-devel] flv decoder question

[Justin Ruggles]

Reimar D?ffinger wrote:
> On Sat, Jul 25, 2009 at 06:43:02PM -0700, Baptiste Coudurier wrote:
>> On 07/25/2009 05:41 PM, Justin Ruggles wrote:
>>> Can someone who knows about the FLV decoder explain these lines in
>>> flv_read_header()?
>>>
>>>      if((flags&  (FLV_HEADER_FLAG_HASVIDEO|FLV_HEADER_FLAG_HASAUDIO))
>>>               != (FLV_HEADER_FLAG_HASVIDEO|FLV_HEADER_FLAG_HASAUDIO))
>>>          s->ctx_flags |= AVFMTCTX_NOHEADER;
>>>
>>> I ask because it is causing av_find_stream_info() to read way more data
>>> than necessary for audio-only FLV files containing Speex.
>> The idea is to read packets to create streams if flags is wrongly set,  
>> but the check will cause NOHEADER to be always be set audio only or  
>> video only files.
> 
> I think this is intentional, since IIRC there are files with audio and video
> where the header still indicates only one of them.

Ok, well I guess that's the price of supporting broken files...  I was
just curious.

Thanks,
Justin