diff options
author | Paul Davis <paul@linuxaudiosystems.com> | 2015-03-31 12:31:06 -0400 |
---|---|---|
committer | Paul Davis <paul@linuxaudiosystems.com> | 2015-03-31 12:31:06 -0400 |
commit | d6574c9c404d5ef26f7b31a8ca12304048d80301 (patch) | |
tree | c3d5d3a75190389214396621f87f32773be24c7d /gtk2_ardour/route_ui.cc | |
parent | 207ac16731baec44c939d85f25c8965d925d7983 (diff) |
remove duplicated calls to update solo/mute state from RouteUI descendants.
RouteUI::set_route() already does the right thing. Also remove RouteUI::mute_changed() since its
only role was to handle the mute change signal from a route, which boost::bind() makes unnecessary
since we can connect update_mute_display() directly.
Diffstat (limited to 'gtk2_ardour/route_ui.cc')
-rw-r--r-- | gtk2_ardour/route_ui.cc | 8 |
1 files changed, 1 insertions, 7 deletions
diff --git a/gtk2_ardour/route_ui.cc b/gtk2_ardour/route_ui.cc index f038ead366..09026e6b29 100644 --- a/gtk2_ardour/route_ui.cc +++ b/gtk2_ardour/route_ui.cc @@ -244,7 +244,7 @@ RouteUI::set_route (boost::shared_ptr<Route> rp) solo_button->set_controllable (_route->solo_control()); _route->active_changed.connect (route_connections, invalidator (*this), boost::bind (&RouteUI::route_active_changed, this), gui_context()); - _route->mute_changed.connect (route_connections, invalidator (*this), boost::bind (&RouteUI::mute_changed, this, _1), gui_context()); + _route->mute_changed.connect (route_connections, invalidator (*this), boost::bind (&RouteUI::update_mute_display, this), gui_context()); _route->comment_changed.connect (route_connections, invalidator (*this), boost::bind (&RouteUI::comment_changed, this, _1), gui_context()); @@ -1186,12 +1186,6 @@ RouteUI::solo_changed_so_update_mute () update_mute_display (); } -void -RouteUI::mute_changed(void* /*src*/) -{ - update_mute_display (); -} - ActiveState RouteUI::mute_active_state (Session* s, boost::shared_ptr<Route> r) { |