From: Pierre Habouzit <madcoder@debian.org>
Date: Sat, 18 Oct 2008 15:54:37 GMT
Subject: Re: [PATCH v2] parse-opt: migrate builtin-checkout-index.
Message-ID: <20081018155436.GA4803@artemis>
In-Reply-To: <7v63nqg4f4.fsf@gitster.siamese.dyndns.org>

On Fri, Oct 17, 2008 at 11:58:23PM +0000, Junio C Hamano wrote:
> Miklos Vajna <vmiklos@frugalware.org> writes:
> 
> > +static int option_parse_z(const struct option *opt,
> > +			  const char *arg, int unset)
> > +{
> > +	line_termination = unset;
> > +	return 0;
> > +}
> > ...
> > +		{ OPTION_CALLBACK, 'z', NULL, NULL, NULL,
> > +			"paths are separated with NUL character",
> > +			PARSE_OPT_NOARG, option_parse_z },
> 
> This adds a new feature to say --no-z from the command line, doesn't it?
> And I suspect the feature is broken ;-).

No it doesn't, --no-foo is only active for long options.

-- 
·O·  Pierre Habouzit
··O                                                madcoder@debian.org
OOO                                                http://www.madism.org