[MPlayer-dev-eng] Re: [PATCH] Profiles support for MEncoder - Upgrade

[Dominik 'Rathann' Mierzejewski]

On Sunday, 22 January 2006 at 13:02, Alban Bedel wrote:
> On Wed, 11 Jan 2006 09:21:03 +0100
> Danny - liste <pi at digitalfantasy.it> wrote:
> 
> > 
> > I update the patch:
> > 
> >     . The default file name is mencoder-profile istead of mencoder.pro
> > 
> >     . I change all the printf() in mp_msg() with class cfgparser and 
> > according level (info, error or fatal) and i put all the commented 
> > printf() on debug level 2 (simple debug) and 3 (dump of all lines read).
> > 
> >     For the question about the exit from the program in case of error 
> > seems that, in this kind of option handling (user function) the return 
> > value is simply ignored.
> > 
> >     I'll investigate ASAP.
> 
> Sorry for jumping in a bit late. The patch look good, but i must say i don't
> like very much the idea of having yet another parser. For the devs it's
> one more to maintain and for the users it's one more sligthly different
> config file format.
> So i'm suggestinge something very similar but extending the config file
> parser instead. With the attached patch you can add some "sections" to
> your config file to define some profile. For example:
> 
> [mpeg4]
> ovc=lavc=1
> lavcopts=vcodec=mpeg4:vbitrate=1200
> 
> [filter640]
> vf=spp,scale=640:480
> 
> Then use with: mencoder -profile mpeg4,filter640 file.avi

I haven't read the code, but is it possible to have some
additional default options after the profiles? How does it
recognize the end of a profile?

R.

-- 
MPlayer RPMs maintainer: http://rpm.greysector.net/mplayer/
"I am Grey. I stand between the candle and the star. We are Grey.
 We stand between the darkness ... and the light."
        -- Delenn in Grey Council in Babylon 5:"Babylon Squared"