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

[Daniel Burrows]

Author: dburrows
Date: Wed Sep 21 21:03:51 2005
New Revision: 4154

Modified:
   branches/aptitude-0.3/aptitude/ChangeLog
   branches/aptitude-0.3/aptitude/src/ui.cc
   branches/aptitude-0.3/aptitude/src/ui.h
Log:
Signal package_states_changed (isn't that redundant?) from apply_current_solutions.

Modified: branches/aptitude-0.3/aptitude/ChangeLog
==============================================================================
--- branches/aptitude-0.3/aptitude/ChangeLog	(original)
+++ branches/aptitude-0.3/aptitude/ChangeLog	Wed Sep 21 21:03:51 2005
@@ -1,5 +1,11 @@
 2005-09-21  Daniel Burrows  <dburrows at debian.org>
 
+	* src/ui.cc, src/ui.h:
+
+	  Trigger package_states_changed from apply_current_solution.
+	  Among other things, this means that the information area is
+	  properly updated when a solution is applied.
+
 	* src/solution_screen.cc:
 
 	  Destroy the whole examiner when there aren't any broken

Modified: branches/aptitude-0.3/aptitude/src/ui.cc
==============================================================================
--- branches/aptitude-0.3/aptitude/src/ui.cc	(original)
+++ branches/aptitude-0.3/aptitude/src/ui.cc	Wed Sep 21 21:03:51 2005
@@ -1576,7 +1576,10 @@
 	}
 
       if(!undo->empty())
-	apt_undos->add_item(undo);
+	{
+	  apt_undos->add_item(undo);
+	  package_states_changed();
+	}
       else
 	delete undo;
     }

Modified: branches/aptitude-0.3/aptitude/src/ui.h
==============================================================================
--- branches/aptitude-0.3/aptitude/src/ui.h	(original)
+++ branches/aptitude-0.3/aptitude/src/ui.h	Wed Sep 21 21:03:51 2005
@@ -39,11 +39,16 @@
 // File menu
 extern sigc::signal0<void> file_quit;
 
-// A signal that widgets watching package states should update themselves.
-//
-// This should be triggered at the end of any keystroke (or mouse-stroke)
-// handler that alters the package states.  At a minimum, it will
-// call vscreen_update().
+/* A signal that widgets watching package states should update themselves.
+ *
+ * This should be triggered at the end of any keystroke (or mouse-stroke)
+ * handler that alters the package states.  At a minimum, it will
+ * call vscreen_update().
+ *
+ * \todo this seems to be at least partly redundant with the similar
+ * signal on the apt cache file.  Should one of them be removed or
+ * scaled back?
+ */
 extern sigc::signal0<void> package_states_changed;
 
 /** Used to determine whether a target is available for the "package actions".