[MPlayer-dev-eng] [PATCH] OSD menu support set multiple commands

Ulion ulion2002 at gmail.com
Mon Dec 10 14:38:29 CET 2007


2007/12/10, Alban Bedel <albeu at free.fr>:
> On Sun, 9 Dec 2007 19:38:28 +0800
> Ulion <ulion2002 at gmail.com> wrote:
>
> > Hello,
> >
> > Here's a patch to make osd menu support multi-commands set in
> > menu.conf.
> >
> > All properties in different tags of menu.conf currently support set
> > mplayer command will support set multi-commands with this patch.
> >
> > Just add break-line between your commands.
> >
> > Indeed in the xml property value, use ';' as seperator will look
> > better for me, but that may break command want to use ';' in it. So
> > use break-line is clean, just looks not so beautiful. But after align
> > following commands with the first command, it looks acceptable for me.
> >
> > If there's no object, I will apply this patch after 2 days. ( To make
> > this patch work properly need other two patches I posted earlier today
> > for input parse fix and a new function for parse and queue cmds. )
>
> Look fine. That's really great, it was badly needed since a looong time.

lol, then I will apply it after I commit the parse and queue commands
patch for input.

-- 
Ulion



More information about the MPlayer-dev-eng mailing list