|
From: <sv...@va...> - 2008-02-18 02:16:18
|
Author: sewardj
Date: 2008-02-18 02:16:22 +0000 (Mon, 18 Feb 2008)
New Revision: 7424
Log:
Fix distinctly bogus comparison routine which caused the new quicksort
implementation to loop forever.
Modified:
branches/DATASYMS/memcheck/mc_leakcheck.c
Modified: branches/DATASYMS/memcheck/mc_leakcheck.c
===================================================================
--- branches/DATASYMS/memcheck/mc_leakcheck.c 2008-02-18 01:59:33 UTC (rev 7423)
+++ branches/DATASYMS/memcheck/mc_leakcheck.c 2008-02-18 02:16:22 UTC (rev 7424)
@@ -215,7 +215,9 @@
{
MC_Chunk* mc1 = *(MC_Chunk**)n1;
MC_Chunk* mc2 = *(MC_Chunk**)n2;
- return (mc1->data < mc2->data ? -1 : 1);
+ if (mc1->data < mc2->data) return -1;
+ if (mc1->data > mc2->data) return 1;
+ return 0;
}
/* If ptr is pointing to a heap-allocated block which hasn't been seen
|