From: Sverre Rabbelier <alturin@gmail.com>
Date: Thu, 24 Jul 2008 20:35:05 GMT
Subject: Re: [PATCH 7/9] builtin-checkout-index.c: use parse_options()
Message-ID: <bd6139dc0807241335i3ab5280aq6a46325428ccc70f@mail.gmail.com>
In-Reply-To: <200807242208.37192.barra_cuda@katamail.com>

On Thu, Jul 24, 2008 at 10:08 PM, Michele Ballabio
<barra_cuda@katamail.com> wrote:
> On Thursday 24 July 2008, Johannes Schindelin wrote:
>> On Wed, 23 Jul 2008, Michele Ballabio wrote:
>>
>> > +           { OPTION_CALLBACK, 'f', "force", &state, NULL,
>> > +             "force overwrite of existing files",
>> > +             PARSE_OPT_NOARG, parse_state_force_cb, 0 },
>>
>> I wonder if this could not be written as
>>
>>               OPT_BOOLEAN('f', "force", &state.force,
>>                       "force overwrite of existing files"),
>
> I did it that way because 'force' is a bitfield.

I thought there is an OPT_BIT?

-- 
Cheers,

Sverre Rabbelier