[Aptitude-devel] r2948 - in branches/aptitude-0.3/aptitude: . src/generic/problemresolver

Daniel Burrows dburrows@costa.debian.org
Fri, 08 Apr 2005 02:51:21 +0000


Author: dburrows
Date: Fri Apr  8 02:51:18 2005
New Revision: 2948

Modified:
   branches/aptitude-0.3/aptitude/ChangeLog
   branches/aptitude-0.3/aptitude/src/generic/problemresolver/test.cc
Log:
Fix totally b0rked logic in dummy_dep::broken().

Modified: branches/aptitude-0.3/aptitude/ChangeLog
==============================================================================
--- branches/aptitude-0.3/aptitude/ChangeLog	(original)
+++ branches/aptitude-0.3/aptitude/ChangeLog	Fri Apr  8 02:51:18 2005
@@ -2,6 +2,16 @@
 
 	* src/generic/problemresolver/test.cc:
 
+	  Fix a rather silly mistake that caused dummy_dep::broken()
+	  to return incorrect answers.
+
+	* src/generic/problemresolver/test.cc:
+
+	  Fix the broken_dep_iter -- it would always claim the
+	  first thing in the list was broken.
+
+	* src/generic/problemresolver/problemresolver.h:
+
 	  Add yet more debugging code.
 
 	* src/generic/problemresolver/test.cc:

Modified: branches/aptitude-0.3/aptitude/src/generic/problemresolver/test.cc
==============================================================================
--- branches/aptitude-0.3/aptitude/src/generic/problemresolver/test.cc	(original)
+++ branches/aptitude-0.3/aptitude/src/generic/problemresolver/test.cc	Fri Apr  8 02:51:18 2005
@@ -275,9 +275,9 @@
 
     for(solver_iterator i=solvers_begin(); i!=solvers_end(); ++i)
       if((*i)->get_package().current_version() == **i)
-	return true;
+	return false;
 
-    return false;
+    return true;
   }
 };