[vlc-devel] Re: CVS Commit (titer)

[Eric Petit]

On Thu, May 08, 2003, Gildas Bazin wrote:
> I would prefer if you kept the old boolean "translation" config option and 
> added a new one for your purpose (e.g. translation-language). This for 2 
> reasons:
>
> 1 - Disabling the translation with a boolean option is more user-friendly 
> than setting the language to "C".
>
> 2 - Changing the type of a config variable breaks the compatibility with 
> current config files. People who already have a config file will have to 
> delete it or explicitly change the "translation" value otherwise 
> translation won't work anymore for them (0 or 1 isn't a valid language).

 I don't agree, having several config variables is not a user-friendly
solution :p Moreover, VLC has not been released since the option exists
and I guess many people that could be annoyed are not end-users and/or
are reading the mailing list, so I don't think it is a big deal.

 I guess the right thing to do would be to use a string_from_list, with
a list containing all the available translations (including 'none').

-- 
Eric Petit <titer at videolan.org>
-- 
This is the vlc-devel mailing-list, see http://www.videolan.org/vlc/
To unsubscribe, please read http://developers.videolan.org/lists.html
If you are in trouble, please contact <postmaster at videolan.org>