[MPlayer-dev-eng] [PATCH] libmenu subtitles selector
Alban Bedel
albeu at free.fr
Tue May 8 01:36:34 CEST 2007
On Tue, 8 May 2007 01:01:35 +0200
Alban Bedel <albeu at free.fr> wrote:
> On Tue, 08 May 2007 00:37:08 +0200
> Benjamin Zores <ben at geexbox.org> wrote:
>
> > Hi,
> >
> > Attached patch adds a new subtitle selector menu to libmenu.
> > It currently only works with DVDs, though it'll be extended to handle
> > Ogg and Matroska in a near future.
> >
> > The /etc/menu.conf sample will be extended afterwards, but to test it
> > you need to add:
> > <subsel name="sub_sel" title="Select subtitle"/>
> > <e name="Subtitle selection ..." ok="set_menu sub_sel"/>
> >
> > Ok to apply ? (Alban ? ;-))
>
> I'm wondering what the point is, isn't the preference menu enouth ?
> If not i would prefer to see a generic solution, that will also
> do for audio and video.
To be more explicit, a cleaner solution would be to add a new property
control to get the list of choice from the current sub property. This
way it could be used for all properties that give a list of choice.
Yeah, i hate too specfic code that could very easily be generic :)
Albeu
PS: Could you post all the pending patch at once (or at least a short
description), not for real review but to get an overall look as
i'm planning a few stuff myself (like making that metadata stuff
beeing able to use any meta instead of a selected list).
More information about the MPlayer-dev-eng
mailing list