Jeff King <peff@xxxxxxxx> writes: > but it also seems like the top bit of that for loop is boilerplate, too: > >> for (;;) { >> switch (parse_options_step(&ctx, options, shortlog_usage)) { >> case PARSE_OPT_HELP: >> exit(129); >> case PARSE_OPT_DONE: >> goto parse_done; >> } Another thing we should be able to refactor is the option help formatter. Cf. 071438a (Teach git-merge -X<option> again., 2008-07-09) -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html