Re: [PATCH 13/14] revert: Introduce --continue to continue the operation

Hi Junio,

Junio C Hamano writes:

Ramkumar Ramachandra [off-list ref] writes:

quoted

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.

True.  I'll try to figure something out before the next iteration.

quoted

+static void verify_opt_mutually_compatible(const char *me, ...)
+{

Isn't "being compatible" by definition "mutual"?

I.e. verify-option-compatibility perhaps?

There's already a verify_opt_compatible which checks that the first
option supplied is compatible with all the other options.  This one
checks that all the the options all the options are compatible with
each other.  What names do you suggest?

-- Ram