git checkout: do not allow switching to a tree-ish that is not a commit
[gitweb.git] / builtin-merge-base.c
index 1cb2925d2f3dc6dc1a38c31bcec1c10ae8146a22..3382b1382a7dcbd525126a35209072da4b4d8041 100644 (file)
@@ -22,10 +22,23 @@ static int show_merge_base(struct commit *rev1, struct commit *rev2, int show_al
 static const char merge_base_usage[] =
 "git merge-base [--all] <commit-id> <commit-id>";
 
+static struct commit *get_commit_reference(const char *arg)
+{
+       unsigned char revkey[20];
+       struct commit *r;
+
+       if (get_sha1(arg, revkey))
+               die("Not a valid object name %s", arg);
+       r = lookup_commit_reference(revkey);
+       if (!r)
+               die("Not a valid commit name %s", arg);
+
+       return r;
+}
+
 int cmd_merge_base(int argc, const char **argv, const char *prefix)
 {
        struct commit *rev1, *rev2;
-       unsigned char rev1key[20], rev2key[20];
        int show_all = 0;
 
        git_config(git_default_config, NULL);
@@ -40,13 +53,8 @@ int cmd_merge_base(int argc, const char **argv, const char *prefix)
        }
        if (argc != 3)
                usage(merge_base_usage);
-       if (get_sha1(argv[1], rev1key))
-               die("Not a valid object name %s", argv[1]);
-       if (get_sha1(argv[2], rev2key))
-               die("Not a valid object name %s", argv[2]);
-       rev1 = lookup_commit_reference(rev1key);
-       rev2 = lookup_commit_reference(rev2key);
-       if (!rev1 || !rev2)
-               return 1;
+       rev1 = get_commit_reference(argv[1]);
+       rev2 = get_commit_reference(argv[2]);
+
        return show_merge_base(rev1, rev2, show_all);
 }