From: Jonathan Nieder <jrnieder@gmail.com>
Date: Wed, 06 Jul 2011 10:25:09 GMT
Subject: Re: [PATCH 13/14] revert: Introduce --continue to continue the operation
Message-ID: <20110706102509.GJ15682@elie>
In-Reply-To: <1309938868-2028-14-git-send-email-artagnon@gmail.com>

Ramkumar Ramachandra wrote:

> --- a/builtin/revert.c
> +++ b/builtin/revert.c
> @@ -47,6 +47,7 @@ struct replay_opts {
>  	enum replay_action action;
>
>  	int reset;
> +	int contin;

Maybe:

	int just_remove_state;
	int resume;
	int abort;

Or:

	enum replay_subcommand {
		REPLAY_RESET,
		REPLAY_CONTINUE,
		REPLAY_ABORT
	};

	enum replay_subcommand subcommand;

Or perhaps this does not need to be part of the replay_opts struct but
can be communicated by which API function gets called (e.g., via
parse_args returning an "enum replay_subcommand").  I dunno.