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

[Daniel Burrows]

Author: dburrows
Date: Sat Sep 24 05:23:00 2005
New Revision: 4226

Modified:
   branches/aptitude-0.3/aptitude/ChangeLog
   branches/aptitude-0.3/aptitude/src/generic/resolver_manager.cc
Log:
Always unsuspend a background thread before calling state_changed().

Modified: branches/aptitude-0.3/aptitude/ChangeLog
==============================================================================
--- branches/aptitude-0.3/aptitude/ChangeLog	(original)
+++ branches/aptitude-0.3/aptitude/ChangeLog	Sat Sep 24 05:23:00 2005
@@ -2,6 +2,11 @@
 
 	* src/generic/resolver_manager.cc:
 
+	  For safety, always unsuspend a background thread before calling
+	  state_changed().
+
+	* src/generic/resolver_manager.cc:
+
 	  Fix background_suspender::unsuspend -- it wasn't clearing its
 	  suspended flag, so the suspender would try to unsuspend AGAIN
 	  when it was destroyed.

Modified: branches/aptitude-0.3/aptitude/src/generic/resolver_manager.cc
==============================================================================
--- branches/aptitude-0.3/aptitude/src/generic/resolver_manager.cc	(original)
+++ branches/aptitude-0.3/aptitude/src/generic/resolver_manager.cc	Sat Sep 24 05:23:00 2005
@@ -722,6 +722,7 @@
     undos->add_item(undo);
 
   l.release();
+  bs.unsuspend();
   state_changed();
 }
 
@@ -744,6 +745,7 @@
   out_of_solutions.put(false);
 
   l.release();
+  bs.unsuspend();
   state_changed();
 }
 
@@ -770,6 +772,7 @@
     undos->add_item(undo);
 
   l.release();
+  bs.unsuspend();
   state_changed();
 }
 
@@ -791,6 +794,7 @@
   out_of_solutions.put(false);
 
   l.release();
+  bs.unsuspend();
   state_changed();
 }
 
@@ -817,6 +821,7 @@
     undos->add_item(undo);
 
   l.release();
+  bs.unsuspend();
   state_changed();
 }
 
@@ -838,6 +843,7 @@
   out_of_solutions.put(false);
 
   l.release();
+  bs.unsuspend();
   state_changed();
 }
 
@@ -864,6 +870,7 @@
     undos->add_item(undo);
 
   l.release();
+  bs.unsuspend();
   state_changed();
 }
 
@@ -885,6 +892,7 @@
   out_of_solutions.put(false);
 
   l.release();
+  bs.unsuspend();
   state_changed();
 }