diff options
author | Phillip Wood <phillip.wood@dunelm.org.uk> | 2021-09-21 10:24:06 +0000 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2021-09-22 12:04:52 -0700 |
commit | 1d188263e0847fd356d7ff245c32e8ebb29115b3 (patch) | |
tree | 9ed10365f9b76114f9fa6e2ab9858b1acf215bd6 /builtin/rebase.c | |
parent | rebase: use our standard error return value (diff) | |
download | tgif-1d188263e0847fd356d7ff245c32e8ebb29115b3.tar.xz |
rebase: use lookup_commit_reference_by_name()
peel_committish() appears to have been copied from the scripted rebase
but it duplicates the functionality of
lookup_commit_reference_by_name() so lets use that instead.
Signed-off-by: Phillip Wood <phillip.wood@dunelm.org.uk>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'builtin/rebase.c')
-rw-r--r-- | builtin/rebase.c | 17 |
1 files changed, 4 insertions, 13 deletions
diff --git a/builtin/rebase.c b/builtin/rebase.c index 8c810d4959..e89e21d0f2 100644 --- a/builtin/rebase.c +++ b/builtin/rebase.c @@ -763,17 +763,6 @@ static int finish_rebase(struct rebase_options *opts) return ret; } -static struct commit *peel_committish(const char *name) -{ - struct object *obj; - struct object_id oid; - - if (get_oid(name, &oid)) - return NULL; - obj = parse_object(the_repository, &oid); - return (struct commit *)peel_to_type(name, 0, obj, OBJ_COMMIT); -} - static void add_var(struct strbuf *buf, const char *name, const char *value) { if (!value) @@ -1846,7 +1835,8 @@ int cmd_rebase(int argc, const char **argv, const char *prefix) if (!strcmp(options.upstream_name, "-")) options.upstream_name = "@{-1}"; } - options.upstream = peel_committish(options.upstream_name); + options.upstream = + lookup_commit_reference_by_name(options.upstream_name); if (!options.upstream) die(_("invalid upstream '%s'"), options.upstream_name); options.upstream_arg = options.upstream_name; @@ -1889,7 +1879,8 @@ int cmd_rebase(int argc, const char **argv, const char *prefix) options.onto = lookup_commit_or_die(&merge_base, options.onto_name); } else { - options.onto = peel_committish(options.onto_name); + options.onto = + lookup_commit_reference_by_name(options.onto_name); if (!options.onto) die(_("Does not point to a valid commit '%s'"), options.onto_name); |