Ramkumar Ramachandra <artagnon@xxxxxxxxx> writes: > diff --git a/builtin/revert.c b/builtin/revert.c > index 9e18d64..6ef56ee 100644 > --- a/builtin/revert.c > +++ b/builtin/revert.c > @@ -47,6 +47,7 @@ struct replay_opts { > enum replay_action action; > > int reset; > + int contin; As Jonathan mentiond, these three look like all action's to me. > +static void verify_opt_mutually_compatible(const char *me, ...) > +{ Isn't "being compatible" by definition "mutual"? I.e. verify-option-compatibility perhaps? -- 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