[PATCH] Improve git-rev-list memory usage further

diff --git a/fetch.c b/fetch.c
index e874538ccc14c58dfaa279d8957d309037a6c388..6ee61616dcd127cb97c1c39c401cdb51b953bddd 100644 (file)
--- a/fetch.c
+++ b/fetch.c
@@ -62,11 +62,22 @@ static int process_tree(struct tree *tree)
        return 0;
 }
 
+#define COMPLETE       1U
+static struct commit_list *complete = NULL;
+
 static int process_commit(struct commit *commit)
 {
        if (parse_commit(commit))
                return -1;
 
+       while (complete && complete->item->date >= commit->date) {
+               pop_most_recent_commit(&complete, COMPLETE);
+       }
+               
+
+       if (commit->object.flags & COMPLETE)
+               return 0;
+
        memcpy(current_commit_sha1, commit->object.sha1, 20);
 
        if (get_tree) {
@@ -78,8 +89,6 @@ static int process_commit(struct commit *commit)
        if (get_history) {
                struct commit_list *parents = commit->parents;
                for (; parents; parents = parents->next) {
-                       if (has_sha1_file(parents->item->object.sha1))
-                               continue;
                        if (process(parents->item->object.sha1,
                                    commit_type))
                                return -1;
@@ -126,6 +135,7 @@ static int process_object(struct object *obj)
 static int process(unsigned char *sha1, const char *type)
 {
        struct object *obj = lookup_object_type(sha1, type);
+
        if (has_sha1_file(sha1)) {
                parse_object(sha1);
                /* We already have it, so we should scan it now. */
@@ -179,18 +189,30 @@ static int interpret_target(char *target, unsigned char *sha1)
        return -1;
 }
 
+static int mark_complete(const char *path, const unsigned char *sha1)
+{
+       struct commit *commit = lookup_commit_reference_gently(sha1, 1);
+       if (commit) {
+               commit->object.flags |= COMPLETE;
+               insert_by_date(commit, &complete);
+       }
+       return 0;
+}
 
 int pull(char *target)
 {
        unsigned char sha1[20];
        int fd = -1;
 
+       save_commit_buffer = 0;
        if (write_ref && current_ref) {
                fd = lock_ref_sha1(write_ref, current_ref);
                if (fd < 0)
                        return -1;
        }
 
+       for_each_ref(mark_complete);
+
        if (interpret_target(target, sha1))
                return error("Could not interpret %s as something to pull",
                             target);