Re: [PATCH 7/9] builtin-checkout-index.c: use parse_options()
- From
- Sverre Rabbelier <alturin@gmail.com>
- Date
- Jul 24, 2008, 20:35 UTC
- 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:
Show 13 quoted lines
> 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