[MPlayer-advusers] Re: Mencoder + lavf == BOOM !

[Nico Sabbi]

Reimar Döffinger wrote:

>Hi,
>On Sun, Oct 02, 2005 at 11:51:11AM +0200, Nico Sabbi wrote:
>  
>
>>http://www1.mplayerhq.hu/cgi-bin/cvsweb.cgi/main/libmpcodecs/vf.c.diff?r1=1.111&r2=1.112
>>
>>this is the patch that broke muxer_lavf.
>>    
>>
>
>What makes you think that?
>
>  
>
>>vf_config isn't being called the first time (before 
>>muxer->fix_parameters()),
>>so video width and height aren't set yet.
>>    
>>
>
>Well, it wasn't before either from what I can see. At least the wrapper
>function in that patch isn't called before the crash.
>
>  
>

uhm, you're right. Do you remember when the call to ve->config()
was moved later on? Previously it was surely called  before 
mux->fix_parameters()
because muxer_lavf had the width and height fields filled in with 
correct data.

Thanks,
    Nico