From fc66d213f8b2f13b9ffd643f01de25ddc95e0972 Mon Sep 17 00:00:00 2001 From: Junio C Hamano Date: Sun, 12 Mar 2006 20:32:06 -0800 Subject: [PATCH] diffcore-delta: tweak hashbase value. This tweaks the maximum hashvalue we use to hash the string into without making the maximum size of the hashtable can grow from the current limit. With this, the renames detected becomes a bit more precise without incurring additional paging cost. Signed-off-by: Junio C Hamano --- diffcore-delta.c | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/diffcore-delta.c b/diffcore-delta.c index f8a751837e..835d82c993 100644 --- a/diffcore-delta.c +++ b/diffcore-delta.c @@ -26,12 +26,19 @@ /* Wild guess at the initial hash size */ #define INITIAL_HASH_SIZE 9 -#define HASHBASE 65537 /* next_prime(2^16) */ + /* We leave more room in smaller hash but do not let it * grow to have unused hole too much. */ #define INITIAL_FREE(sz_log2) ((1<<(sz_log2))*(sz_log2-3)/(sz_log2)) +/* A prime rather carefully chosen between 2^16..2^17, so that + * HASHBASE < INITIAL_FREE(17). We want to keep the maximum hashtable + * size under the current 2<<17 maximum, which can hold this many + * different values before overflowing to hashtable of size 2<<18. + */ +#define HASHBASE 107927 + struct spanhash { unsigned long hashval; unsigned long cnt;