From: Michele Ballabio <barra_cuda@katamail.com>
Date: Thu, 24 Jul 2008 20:08:37 GMT
Subject: Re: [PATCH 7/9] builtin-checkout-index.c: use parse_options()
Message-ID: <200807242208.37192.barra_cuda@katamail.com>
In-Reply-To: <alpine.DEB.1.00.0807241543190.8986@racer>

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.