From 419006dd834827f22be9a73753c2355214b3cdc9 Mon Sep 17 00:00:00 2001 From: Paul Davis Date: Tue, 21 Feb 2017 18:45:16 +0100 Subject: PBD::Signal<...>::connect() is already thread safe, so drop intermediate proxy/call_slot() invocation in handling Controllable::Changed within AutomationController --- gtk2_ardour/automation_controller.cc | 8 +------- gtk2_ardour/automation_controller.h | 2 -- 2 files changed, 1 insertion(+), 9 deletions(-) (limited to 'gtk2_ardour') diff --git a/gtk2_ardour/automation_controller.cc b/gtk2_ardour/automation_controller.cc index 7411155551..056e36a778 100644 --- a/gtk2_ardour/automation_controller.cc +++ b/gtk2_ardour/automation_controller.cc @@ -120,7 +120,7 @@ AutomationController::AutomationController(boost::shared_ptr _screen_update_connection = Timers::rapid_connect ( sigc::mem_fun (*this, &AutomationController::display_effective_value)); - ac->Changed.connect (_changed_connection, invalidator (*this), boost::bind (&AutomationController::value_changed, this), gui_context()); + ac->Changed.connect (_changed_connection, invalidator (*this), boost::bind (&AutomationController::display_effective_value, this), gui_context()); add(*_widget); show_all(); @@ -334,12 +334,6 @@ AutomationController::on_button_release(GdkEventButton* ev) return false; } -void -AutomationController::value_changed () -{ - Gtkmm2ext::UI::instance()->call_slot (invalidator (*this), boost::bind (&AutomationController::display_effective_value, this)); -} - /** Stop updating our value from our controllable */ void AutomationController::stop_updating () diff --git a/gtk2_ardour/automation_controller.h b/gtk2_ardour/automation_controller.h index 4d5826023e..126383d517 100644 --- a/gtk2_ardour/automation_controller.h +++ b/gtk2_ardour/automation_controller.h @@ -93,8 +93,6 @@ private: void set_freq_beats(double beats); bool on_button_release(GdkEventButton* ev); - void value_changed(); - Gtk::Widget* _widget; boost::shared_ptr _printer; boost::shared_ptr _controllable; -- cgit v1.2.3