From: Junio C Hamano <junkio@cox.net>
Date: Fri, 21 Apr 2006 06:36:22 +0000 (-0700)
Subject: pack-objects: do not stop at object that is "too small"
X-Git-Tag: v1.3.1~8
X-Git-Url: https://git.verplant.org/?a=commitdiff_plain;h=f527cb8c38964a90b1b13485f2ad46b72960d387;p=git.git

pack-objects: do not stop at object that is "too small"

Because we sort the delta window by name-hash and then size,
hitting an object that is too small to consider as a delta base
for the current object does not mean we do not have better
candidate in the window beyond it.

Noticed by Shawn Pearce, analyzed by Nico, Linus and me.

Signed-off-by: Junio C Hamano <junkio@cox.net>
---

diff --git a/pack-objects.c b/pack-objects.c
index 09f4f2c9..f7d62175 100644
--- a/pack-objects.c
+++ b/pack-objects.c
@@ -1052,7 +1052,7 @@ static int try_delta(struct unpacked *cur, struct unpacked *old, unsigned max_de
 	if (cur_entry->delta)
 		max_size = cur_entry->delta_size-1;
 	if (sizediff >= max_size)
-		return -1;
+		return 0;
 	delta_buf = diff_delta(old->data, oldsize,
 			       cur->data, size, &delta_size, max_size);
 	if (!delta_buf)