From: Jeff King <peff@peff.net>
Date: Thu, 16 Apr 2009 08:34:43 GMT
Subject: Re: [PATCH] Add the diff option --no-defaults
Message-ID: <20090416083443.GA27399@coredump.intra.peff.net>
In-Reply-To: <alpine.DEB.1.00.0904140036341.10279@pacific.mpi-cbg.de>

On Tue, Apr 14, 2009 at 12:37:42AM +0200, Johannes Schindelin wrote:

> >   1. You could loosen (1) above by assuming that --no-defaults will
> >      never appears as the argument to an option, and therefore any time
> >      you find it, it should be respected. Thus your first parse is just
> >      a simple loop looking for the option.
> 
> I go with 1)

This feels very hack-ish to me, but perhaps this is a case of "perfect
is the enemy of the good".

-Peff