From: Johannes Schindelin <johannes.schindelin@gmx.de>
Date: Thu, 24 Jul 2008 14:44:26 GMT
Subject: Re: [PATCH 7/9] builtin-checkout-index.c: use parse_options()
Message-ID: <alpine.DEB.1.00.0807241543190.8986@racer>
In-Reply-To: <1216849332-26813-8-git-send-email-barra_cuda@katamail.com>

Hi,

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"),

Ciao,
Dscho