[vlc-devel] [PATCH 1/1] codec/dirac: Rewrite libdirac(-research) encoding support

[Laurent Aimar]

On Tue, Nov 11, 2008, David Flynn wrote:
> On 2008-11-11, Laurent Aimar <fenrir at via.ecp.fr> wrote:
> > On Mon, Nov 10, 2008, David Flynn wrote:
> >  The workaround I see, is to use 
> >  - a string list with something like "auto", "on", "off"
> >  - an integer list with -1, 0, 1 and comments.
> >  - an integer as you do but with a range including -1
> >  - a new variable type (a lot of work)
> 
> It looked much simpler to make vlc cope when i was still allowed to use
> var_Get()
 You can use var_Get. But it does not solve the problem anyway.
(var_Get returning -1 for a bool looks like a bug to me)

 A mean to set the variable to "no set" need to be added to the
command line parser and the gui if you want that way.

 I find the solution of an integer with 3 states pretty clean but that
just my opinion.

-- 
fenrir