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

[David Flynn]

On 2008-11-11, Laurent Aimar <fenrir at via.ecp.fr> wrote:
> 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.

It depends -- for some options, the ability to go --no-foo and --foo is
quite intuitive and turns into a checkbox in a gui.  Being able to keep
that and possibly even add --dontcare-foo would be good (that could even
apply to non booleans).

..david