mirror of
https://github.com/git/git
synced 2024-08-24 18:26:02 +00:00
sequencer: lib'ify do_pick_commit()
Instead of dying there, let the caller high up in the callchain notice the error and handle it (by dying, still). The only two callers of do_pick_commit(), pick_commits() and single_pick() already check the return value and pass it on to their callers, so their callers must be already prepared to handle error returns, and with this step, we make it notice an error return from this function. So this is a safe conversion to make do_pick_commit() callable from new callers that want it not to die, without changing the external behaviour of anything existing. While at it, remove the superfluous space. Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de> Signed-off-by: Junio C Hamano <gitster@pobox.com>
This commit is contained in:
parent
c527b55e74
commit
f74087f6ed
|
@ -464,7 +464,7 @@ static int do_pick_commit(struct commit *commit, struct replay_opts *opts)
|
||||||
* to work on.
|
* to work on.
|
||||||
*/
|
*/
|
||||||
if (write_cache_as_tree(head, 0, NULL))
|
if (write_cache_as_tree(head, 0, NULL))
|
||||||
die (_("Your index file is unmerged."));
|
return error(_("Your index file is unmerged."));
|
||||||
} else {
|
} else {
|
||||||
unborn = get_sha1("HEAD", head);
|
unborn = get_sha1("HEAD", head);
|
||||||
if (unborn)
|
if (unborn)
|
||||||
|
|
Loading…
Reference in a new issue