[Aptitude-svn-commit] r4109 - in branches/aptitude-0.3/aptitude: . src/generic

[Daniel Burrows]

Author: dburrows
Date: Sun Sep 18 03:57:14 2005
New Revision: 4109

Modified:
   branches/aptitude-0.3/aptitude/ChangeLog
   branches/aptitude-0.3/aptitude/src/generic/immset.h
Log:
Switch the second comparison of the roots of the two trees in node_contains.

Modified: branches/aptitude-0.3/aptitude/ChangeLog
==============================================================================
--- branches/aptitude-0.3/aptitude/ChangeLog	(original)
+++ branches/aptitude-0.3/aptitude/ChangeLog	Sun Sep 18 03:57:14 2005
@@ -1,5 +1,10 @@
 2005-09-17  Daniel Burrows  <dburrows at debian.org>
 
+	* src/generic/immset.h:
+
+	  Switch the second comparison of the roots of the two trees in
+	  node_contains() -- I forgot to transform that one to its dual.
+
 	* tests/test_wtree.cc:
 
 	  Add a test for the subset (containment) routine.

Modified: branches/aptitude-0.3/aptitude/src/generic/immset.h
==============================================================================
--- branches/aptitude-0.3/aptitude/src/generic/immset.h	(original)
+++ branches/aptitude-0.3/aptitude/src/generic/immset.h	Sun Sep 18 03:57:14 2005
@@ -587,7 +587,7 @@
 	  return node_contains(n1, n2.getLeft(), f) &&
 	         node_contains(n1.getRight(), n2repl, f);
         }
-      else if(value_compare(n1.getVal(), n2.getVal()))
+      else if(value_compare(n2.getVal(), n1.getVal()))
         {
 	  // Strip the right subtree of n2.
 	  node n2repl = n2.getRight().empty()