Don't parse commit objects more than once.

Yes, the "parse_commit()" already checks for this condition, but
we need to check for it in rev-tree too, so that we don't start
walking the parent chain unnecessarily.

diff --git a/rev-tree.c b/rev-tree.c
index c3884e3595cd00f8c1108429060ed195f7506514..ebeceb070b72744b47873c0a562d3ec692269a96 100644 (file)
--- a/rev-tree.c
+++ b/rev-tree.c
@@ -55,6 +55,10 @@ void process_commit(unsigned char *sha1)
 {
        struct commit_list *parents;
        struct commit *obj = lookup_commit(sha1);
+
+       if (obj->object.parsed)
+               return;
+
        parse_commit(obj);
        
        parents = obj->parents;