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

[Alban Bedel]

On Sun, 22 Jan 2006 13:40:49 +0100
Dominik 'Rathann' Mierzejewski <dominik at rangers.eu.org> wrote:

> 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?

Yes, use [default] to switch back to "normal" behaviour.

> How does it recognize the end of a profile?

Start of another profile, so you can't name a profile "default".

A few other things, profiles can be listed with -profile help
and a description can be attached with profile-desc="blah blah"
in the profile section.

	Albeu