From: Jeff King <peff@peff.net>
Date: Fri, 02 Jun 2017 20:15:33 GMT
Subject: Re: [PATCH 2/3] revision.c: use skip_prefix() in handle_revision_opt()
Message-ID: <20170602201533.bw4cmsqsygfdif2o@sigill.intra.peff.net>
In-Reply-To: <20170602201143.6avkhp224no3dukb@sigill.intra.peff.net>

On Fri, Jun 02, 2017 at 04:11:43PM -0400, Jeff King wrote:

>   if (match_opt(arg, "--early-output"), &optarg)) {
> 	int count = optarg ? atoi(optarg) : 100;
> 	...
>   }
> 
> which is a little nicer and could maybe help other options (I didn't see
> any, though).

I take it back. This would help --show-linear-break that your patch also
touches. And maybe --pretty, too.

-Peff