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

Alban Bedel albeu at free.fr
Mon Dec 10 13:02:21 CET 2007


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.

	Albeu




More information about the MPlayer-dev-eng mailing list