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

[Guillaume POIRIER]

Hi,

On 1/22/06, Alban Bedel <albeu at free.fr> 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 second that. The proposed patch may be very powerful (for examples,
the multi-language stuff is neat, yet useless for me), but indeed, the
simpler the patch, the better.

Guillaume
--
A word to the wise ain't necessary - it's the stupid ones that need the advice.
    Bill Cosby

MPlayer's doc isn't up-to-date. Visit my updated mirror here:
http://tuxrip.free.fr//MPlayer-DOCS-HTML/en/
http://tuxrip.free.fr//MPlayer-DOCS-HTML/fr/