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

[Alban Bedel]

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

	Albeu
-------------- next part --------------
A non-text attachment was scrubbed...
Name: profile-alb.diff
Type: application/octet-stream
Size: 7689 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20060122/524508c2/attachment.obj>