Skip to content

Commit

Permalink
git-pack-objects: make "--window=x" semantics more logical.
Browse files Browse the repository at this point in the history
A zero disables delta generation (like before), but we make the window
be one bigger than specified, since we use one entry for the one to be
tested (it used to be that "--window=1" was meaningless, since we'd have
used up the single-entry window with the entry to be tested, and had no
chance of actually ever finding a delta).

The default window remains at 10, but now it really means "test the 10
closest objects", not "test the 9 closest objects".
  • Loading branch information
Linus Torvalds committed Jun 26, 2005
1 parent 75c42d8 commit f846bbf
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions pack-objects.c
Original file line number Diff line number Diff line change
Expand Up @@ -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 {
Expand Down Expand Up @@ -364,6 +363,7 @@ static void find_deltas(struct object_entry **list, int window)
int main(int argc, char **argv)
{
char line[128];
int window = 10;
int i;

for (i = 1; i < argc; i++) {
Expand All @@ -372,7 +372,7 @@ 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;
Expand All @@ -399,8 +399,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)
find_deltas(sorted_by_type, window+1);

write_pack_file();
write_index_file();
Expand Down

0 comments on commit f846bbf

Please sign in to comment.