[vlc-devel] [PATCH 1/1] codec/dirac: Rewrite libdirac(-research) encoding support
David Flynn
davidf+nntp at woaf.net
Tue Nov 11 22:59:02 CET 2008
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
More information about the vlc-devel
mailing list