[MPlayer-dev-eng] mencoder variable FPS

[Nico Sabbi]

Keith Chew wrote:

> Hi Nico
>
>> maybe you didn't read the muxers' code, especially not muxer_avi.c ;)
>> mencoder is a boatload of crap kept together with band aids (it's
>> essentially a frontend for an avi muxer hacked to hell in order to
>> output other file formats); your code is hackish and doesn't fit at
>> all with the structure of mencoder
>>
>
> Your input is not very helpful to me. If you have a suggestion on how
> to make it fit into the structure, I am all ears.
>
> Because Mencoder is designed to run as a single flow of tasks (ie
> initialise, read and write output), most variables are initialised at
> the start. I am running the mencoder process 24x7, and I believe
> extending it to take runtime variable changes is not trivial. 

indeed it's not trivial

> At this
> stage, all I am interested in is the impact of the code added, or
> suggestions on a better place to handle the runtime change of the
> force_fps variable.

it can't work if the output file format doesn't support variable 
framerate (such as avi),
and since mencoder as a whole is very avi-centric I wouldn't be surprised if
other things broke

>
> So far, I have not seen any problems in my testing. It has been
> running smoothly for over 3 days.
>
what output format are you using?