qsort(): ptrdiff_t may be larger than int
authorJunio C Hamano <junkio@cox.net>
Fri, 9 Dec 2005 01:29:11 +0000 (17:29 -0800)
committerJunio C Hamano <junkio@cox.net>
Fri, 9 Dec 2005 01:29:11 +0000 (17:29 -0800)
This is a companion patch to e23eff8be92a2a2cb66b53deef020063cff285ed
commit.  The same logic, the same rationale that a comparison
function that returns an int should not just compute a ptrdiff_t
and return it.

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

index 7d7bcb2..df19e49 100644 (file)
@@ -140,7 +140,12 @@ static int compare_info(const void *a_, const void *b_)
                return 1;
 
        /* then it does not matter but at least keep the comparison stable */
-       return (*a)->p - (*b)->p;
+       if ((*a)->p == (*b)->p)
+               return 0;
+       else if ((*a)->p < (*b)->p)
+               return -1;
+       else
+               return 1;
 }
 
 static void init_pack_info(const char *infofile, int force)