[MPlayer-dev-eng] [PATCH] libmenu subtitles selector

[Nico Sabbi]

Benjamin Zores 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 ? ;-))
> 
> Ben

I'm not particular against it, but I wonder how wise is it
adding code that depends so much on the current subtitle code
(horrid as it is)