PBD::Signal<...>::connect() is already thread safe, so drop intermediate proxy/call_s...

diff --git a/gtk2_ardour/automation_controller.cc b/gtk2_ardour/automation_controller.cc
index 7411155551fb1a6e077f1e01ad6cdd503901715f..056e36a77835dc4048deef775dfda50ef8a51c9e 100644 (file)
--- a/gtk2_ardour/automation_controller.cc
+++ b/gtk2_ardour/automation_controller.cc
@@ -120,7 +120,7 @@ AutomationController::AutomationController(boost::shared_ptr<Automatable>
        _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 4d5826023e35d50c5af9f0d34657d78dcc4a4599..126383d517f22eee25c7dc7be7990f11f03b2327 100644 (file)
--- 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<ARDOUR::Automatable>       _printer;
        boost::shared_ptr<ARDOUR::AutomationControl> _controllable;