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