cherry-pick: refactor commit parsing code

These lines are really just lookup_commit_reference
re-implemented.

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

diff --git a/builtin-revert.c b/builtin-revert.c
index 83e5c0a755846063a11e3c16432a3faa358d1359..012c64644d6ec0f0519721aa35b55c79e439ec81 100644 (file)
--- a/builtin-revert.c
+++ b/builtin-revert.c
@@ -68,15 +68,9 @@ static void parse_args(int argc, const char **argv)
 
        if (get_sha1(arg, sha1))
                die ("Cannot find '%s'", arg);
 
        if (get_sha1(arg, sha1))
                die ("Cannot find '%s'", arg);

-       commit = (struct commit *)parse_object(sha1);
+       commit = lookup_commit_reference(sha1);

        if (!commit)
        if (!commit)

-               die ("Could not find %s", sha1_to_hex(sha1));
-       if (commit->object.type == OBJ_TAG) {
-               commit = (struct commit *)
-                       deref_tag((struct object *)commit, arg, strlen(arg));
-       }
-       if (commit->object.type != OBJ_COMMIT)
-               die ("'%s' does not point to a commit", arg);
+               exit(1);

 }
 
 static char *get_oneline(const char *message)
 }
 
 static char *get_oneline(const char *message)