git-pack-objects: do the delta search in reverse size order
[git.git] / pack-objects.c
index dfa9d44..286e450 100644 (file)
@@ -2,7 +2,7 @@
 #include "object.h"
 #include "delta.h"
 
-static const char pack_usage[] = "git-pack-objects [--window=N] base-name < object-list";
+static const char pack_usage[] = "git-pack-objects [--window=N] [--depth=N] base-name < object-list";
 
 enum object_type {
        OBJ_NONE,
@@ -26,7 +26,6 @@ struct object_entry {
 static struct object_entry **sorted_by_sha, **sorted_by_type;
 static struct object_entry *objects = NULL;
 static int nr_objects = 0, nr_alloc = 0;
-static int delta_window = 10;
 static const char *base_name;
 
 struct myfile {
@@ -83,8 +82,9 @@ static void *delta_against(void *buf, unsigned long size, struct object_entry *e
 
        if (!otherbuf)
                die("unable to read %s", sha1_to_hex(entry->delta->sha1));
-        delta_buf = diff_delta(buf, size, otherbuf, othersize, &delta_size);
-        if (delta_size != entry->delta_size)
+        delta_buf = diff_delta(otherbuf, othersize,
+                              buf, size, &delta_size, ~0UL);
+        if (!delta_buf || delta_size != entry->delta_size)
                die("delta size changed");
         free(buf);
         free(otherbuf);
@@ -96,7 +96,7 @@ static unsigned long write_object(FILE *f, struct object_entry *entry)
        unsigned long size;
        char type[10];
        void *buf = read_sha1_file(entry->sha1, type, &size);
-       char header[21];
+       char header[25];
        unsigned hdrlen, datalen;
 
        if (!buf)
@@ -110,16 +110,16 @@ static unsigned long write_object(FILE *f, struct object_entry *entry)
         * instead.
         */
        header[0] = ".CTB"[entry->type];
-       datalen = htonl(size);
-       memcpy(header+1, &datalen, 4);
        hdrlen = 5;
        if (entry->delta) {
                header[0] = 'D';
-               memcpy(header+1, entry->delta, 20);
+               memcpy(header+5, entry->delta, 20);
                buf = delta_against(buf, size, entry);
                size = entry->delta_size;
-               hdrlen = 21;
+               hdrlen = 25;
        }
+       datalen = htonl(size);
+       memcpy(header+1, &datalen, 4);
        fwrite(header, hdrlen, 1, f);
        datalen = fwrite_compressed(buf, size, f);
        free(buf);
@@ -280,29 +280,34 @@ struct unpacked {
 };
 
 /*
- * We search for deltas in a list sorted by type and by size, and
- * walk the "old" chain backwards in the list, so if the type
- * has changed or the size difference is too big, there's no point
- * in even continuing the walk, since the other old objects are
- * going to be even smaller or of a different type. So return -1
- * once we determine that there's no point even trying.
+ * We search for deltas _backwards_ in a list sorted by type and
+ * by size, so that we see progressively smaller and smaller files.
+ * That's because we prefer deltas to be from the bigger file
+ * to the smaller - deletes are potentially cheaper, but perhaps
+ * more importantly, the bigger file is likely the more recent
+ * one.
  */
-static int try_delta(struct unpacked *cur, struct unpacked *old)
+static int try_delta(struct unpacked *cur, struct unpacked *old, unsigned max_depth)
 {
        struct object_entry *cur_entry = cur->entry;
        struct object_entry *old_entry = old->entry;
-       unsigned long size, oldsize, delta_size;
+       unsigned long size, oldsize, delta_size, sizediff;
+       long max_size;
        void *delta_buf;
 
        /* Don't bother doing diffs between different types */
        if (cur_entry->type != old_entry->type)
                return -1;
 
-       /* Size is guaranteed to be larger than or equal to oldsize */
        size = cur_entry->size;
+       if (size < 50)
+               return -1;
        oldsize = old_entry->size;
-       if (size - oldsize > oldsize / 4)
+       sizediff = oldsize > size ? oldsize - size : size - oldsize;
+       if (sizediff > size / 8)
                return -1;
+       if (old_entry->depth >= max_depth)
+               return 0;
 
        /*
         * NOTE!
@@ -311,28 +316,30 @@ static int try_delta(struct unpacked *cur, struct unpacked *old)
         * more space-efficient (deletes don't have to say _what_ they
         * delete).
         */
-       delta_buf = diff_delta(cur->data, size, old->data, oldsize, &delta_size);
+       max_size = size / 2 - 20;
+       if (cur_entry->delta)
+               max_size = cur_entry->delta_size-1;
+       delta_buf = diff_delta(old->data, oldsize,
+                              cur->data, size, &delta_size, max_size);
        if (!delta_buf)
-               die("unable to create delta");
-       if (delta_size + 20 < size / 2) {
-               if (!cur_entry->delta || cur_entry->delta_size > delta_size) {
-                       cur_entry->delta = old_entry;
-                       cur_entry->delta_size = delta_size;
-               }
-       }
+               return 0;
+       cur_entry->delta = old_entry;
+       cur_entry->delta_size = delta_size;
+       cur_entry->depth = old_entry->depth + 1;
        free(delta_buf);
        return 0;
 }
 
-static void find_deltas(struct object_entry **list, int window)
+static void find_deltas(struct object_entry **list, int window, int depth)
 {
-       unsigned int i;
+       int i, idx;
        unsigned int array_size = window * sizeof(struct unpacked);
        struct unpacked *array = xmalloc(array_size);
 
        memset(array, 0, array_size);
-       for (i = 0; i < nr_objects; i++) {
-               unsigned int idx = i % window;
+       i = nr_objects;
+       idx = 0;
+       while (--i >= 0) {
                struct object_entry *entry = list[i];
                struct unpacked *n = array + idx;
                unsigned long size;
@@ -353,15 +360,19 @@ static void find_deltas(struct object_entry **list, int window)
                        m = array + other_idx;
                        if (!m->entry)
                                break;
-                       if (try_delta(n, m) < 0)
+                       if (try_delta(n, m, depth) < 0)
                                break;
                }
+               idx++;
+               if (idx >= window)
+                       idx = 0;
        }
 }
 
 int main(int argc, char **argv)
 {
        char line[128];
+       int window = 10, depth = 10;
        int i;
 
        for (i = 1; i < argc; i++) {
@@ -370,11 +381,18 @@ int main(int argc, char **argv)
                if (*arg == '-') {
                        if (!strncmp("--window=", arg, 9)) {
                                char *end;
-                               delta_window = strtoul(arg+9, &end, 0);
+                               window = strtoul(arg+9, &end, 0);
                                if (!arg[9] || *end)
                                        usage(pack_usage);
                                continue;
                        }
+                       if (!strncmp("--depth=", arg, 8)) {
+                               char *end;
+                               depth = strtoul(arg+8, &end, 0);
+                               if (!arg[8] || *end)
+                                       usage(pack_usage);
+                               continue;
+                       }
                        usage(pack_usage);
                }
                if (base_name)
@@ -397,8 +415,8 @@ int main(int argc, char **argv)
 
        sorted_by_sha = create_sorted_list(sha1_sort);
        sorted_by_type = create_sorted_list(type_size_sort);
-       if (delta_window)
-               find_deltas(sorted_by_type, delta_window);
+       if (window && depth)
+               find_deltas(sorted_by_type, window+1, depth);
 
        write_pack_file();
        write_index_file();