test-hashmap: use "unsigned int" for hash storage

The hashmap API always use an unsigned value for storing
and comparing hashes. Whereas this test code uses "int".
This works out in practice since one can typically
round-trip between "int" and "unsigned int". But since this
is essentially reference code for the hashmap API, we should
model using the correct types.

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

diff --git a/t/helper/test-hashmap.c b/t/helper/test-hashmap.c
index 56efff36e8d06d57debba44d1ea7d2437a48f18d..9ae9281c071254019ccca3486b3a6762d9c0085f 100644 (file)
--- a/t/helper/test-hashmap.c
+++ b/t/helper/test-hashmap.c
@@ -30,7 +30,8 @@ static int test_entry_cmp(const void *cmp_data,
                return strcmp(e1->key, key ? key : e2->key);
 }
 
                return strcmp(e1->key, key ? key : e2->key);
 }
 

-static struct test_entry *alloc_test_entry(int hash, char *key, char *value)
+static struct test_entry *alloc_test_entry(unsigned int hash,
+                                          char *key, char *value)

 {
        size_t klen = strlen(key);
        size_t vlen = strlen(value);
 {
        size_t klen = strlen(key);
        size_t vlen = strlen(value);


@@ -156,7 +157,7 @@ int cmd_main(int argc, const char **argv)
        /* process commands from stdin */
        while (strbuf_getline(&line, stdin) != EOF) {
                char *cmd, *p1 = NULL, *p2 = NULL;
        /* process commands from stdin */
        while (strbuf_getline(&line, stdin) != EOF) {
                char *cmd, *p1 = NULL, *p2 = NULL;

-               int hash = 0;
+               unsigned int hash = 0;

                struct test_entry *entry;
 
                /* break line into command and up to two parameters */
                struct test_entry *entry;
 
                /* break line into command and up to two parameters */