Fix up route group state signal handling. Make the PropertyList version

of Stateful::set_properties emit a changed signal after it has done its work.


git-svn-id: svn://localhost/ardour2/branches/3.0@6826 d708f5d6-7413-0410-9779-e7cbd77b26cf

1 files changed, 5 insertions, 5 deletions

diff --git a/gtk2_ardour/mixer_ui.cc b/gtk2_ardour/mixer_ui.cc
index 618d1bbe36..c1dc4aff9f 100644
--- a/gtk2_ardour/mixer_ui.cc
+++ b/gtk2_ardour/mixer_ui.cc
@@ -1204,14 +1204,12 @@ Mixer_UI::remove_selected_route_group ()
 }
 
 void
-Mixer_UI::group_flags_changed (void* src, RouteGroup* group)
+Mixer_UI::route_group_property_changed (RouteGroup* group, const PropertyChange& change)
 {
 	if (in_group_row_change) {
 		return;
 	}
 
-	ENSURE_GUI_THREAD (*this, &Mixer_UI::group_flags_changed, src, group)
-
 	/* force an update of any mixer strips that are using this group,
 	   otherwise mix group names don't change in mixer strips
 	*/
@@ -1238,7 +1236,9 @@ Mixer_UI::group_flags_changed (void* src, RouteGroup* group)
 
 	in_group_row_change = false;
 
-	_group_tabs->set_dirty ();
+	if (change.contains (Properties::name)) {
+		_group_tabs->set_dirty ();
+	}
 }
 
 void
@@ -1312,7 +1312,7 @@ Mixer_UI::add_route_group (RouteGroup* group)
 		focus = true;
 	}
 
-	group->FlagsChanged.connect (*this, invalidator (*this), ui_bind (&Mixer_UI::group_flags_changed, this, _1, group), gui_context());
+	group->PropertyChanged.connect (*this, invalidator (*this), ui_bind (&Mixer_UI::route_group_property_changed, this, group, _1), gui_context());
 
 	if (focus) {
 		TreeViewColumn* col = group_display.get_column (0);