From: Jeff King <peff@peff.net>
Date: Tue, 28 Feb 2012 19:09:36 GMT
Subject: Re: [PATCH 3/3 v2] parse-options: remove PARSE_OPT_NEGHELP
Message-ID: <20120228190936.GC11260@sigill.intra.peff.net>
In-Reply-To: <4F4D25A1.8050702@lsrfire.ath.cx>

On Tue, Feb 28, 2012 at 08:06:09PM +0100, René Scharfe wrote:

> PARSE_OPT_NEGHELP is confusing because short options defined with that
> flag do the opposite of what the helptext says. It is also not needed
> anymore now that options starting with no- can be negated by removing
> that prefix. Convert its only two users to OPT_NEGBIT() and OPT_BOOL()
> and then remove support for PARSE_OPT_NEGHELP.
> 
> Signed-off-by: Rene Scharfe <rene.scharfe@lsrfire.ath.cx>
> ---
> This version doesn't invert the logic in grep anymore.

Thanks, I like this one much better.

-Peff