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
-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
Valgrind-developers mailing list
Valgrind-developers@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/valgrind-developers
