Johannes Schindelin <johannes.schindelin@xxxxxx> writes:
> static int pick_commits(struct commit_list *todo_list, struct replay_opts *opts)
> @@ -1128,9 +1130,9 @@ int sequencer_pick_revisions(struct replay_opts *opts)
> return -1;
> if (get_sha1("HEAD", sha1) && (opts->action == REPLAY_REVERT))
> return error(_("Can't revert as initial commit"));
> - if (save_head(sha1_to_hex(sha1)))
> + if (save_head(sha1_to_hex(sha1)) ||
> + save_opts(opts))
> return -1;
> - save_opts(opts);
I think it is much easier to read to keep this on a single line. It
would be more verbose but an even easier would be to keep these two
operations two separate steps, i.e.
if (save_head())
return -1;
if (save_opts())
return -1;
--
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