Matthieu Moy wrote: > - if (!prefixcmp(arg, "--branches=")) { > + if ((argcount = diff_long_opt("branches", argv + i, &optarg))) { > + i += argcount - 1; > struct all_refs_cb cb; As before: is this the right change where the argument is optional? Thanks for a pleasant read. Jonathan -- 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