From: Jeff King Date: Mon, 23 Jun 2008 19:59:07 GMT Subject: Re: [PATCH 1/2] parse_options: Add flag to prevent errors for further processing Message-ID: <20080623195906.GC29569@sigill.intra.peff.net> In-Reply-To: <7v4p7khqp7.fsf@gitster.siamese.dyndns.org> On Mon, Jun 23, 2008 at 12:55:00PM -0700, Junio C Hamano wrote: > >> On Wed, Jun 18, 2008 at 11:52:42AM -0700, Junio C Hamano wrote: > > I believe not. I think that Junio prefers some callback that can handle a > > whole bunch of options (as opposed to the callback we can have now, to > > handle arguments for a specific option). > > Sorry, no. I do not want callbacks. I've been saying that parser > cascading is easier if you use an incremental interface like diff option > parser does. Now I'm confused: my understanding is that the diff option parser just leaves unrecognized stuff in argv. But isn't that what a PARSE_OPTIONS_IGNORE_UNKNOWN flag would do, and isn't that wrong? -Peff