[FFmpeg-devel] BFI Demuxer

[Vitor Sessak]

Hi

Sisir Koppaka wrote:
> Made most of the suggested changes...pts is also set OK now and both audio
> and video streams are playing...(video decoder needs some more work
> though)...

As I've already said, if the demuxer is finished, you _don't_ have to 
finish the decoder to get it reviewed and commited. The demuxer and the 
decoder are two independent things. It will be much easier for you to do 
one thing at a time.

> 
>>>> +static int bfi_read_close(AVFormatContext * s)
> 
>>>>> +{
>>>>> +    BFIContext *bfi = s->priv_data;
>>>>> +    av_free(s->streams[bfi->video_index]->codec->extradata);
>>>> this is wrong
>>>>
>>> Should we free extradata in the decoder close function in that case?
>> see av_close_input_stream()
>>
>  I couldn't locate this function here -
> http://www.irisa.fr/texmex/people/dufouil/ffmpegdoxy/globals_func_0x61.html#index_a

Why are you searching it at google? You should search the ffmpeg source 
code in your computer. I'd suggest

vitor at vitor-laptop:~/ffmpeg$ grep av_close_input_stream */*.{c,h}

see where this function is defined and implemented (and read the 
comments there). If even then you don't understand what it does, try to 
understand the source code.

-Vitor