Update ghost note presence when mouse mode is changed. Fixes #3559.

[ardour.git] / gtk2_ardour / automation_controller.cc

diff --git a/gtk2_ardour/automation_controller.cc b/gtk2_ardour/automation_controller.cc
index 6f62608f0a8c789f067a3bc1ae897841c1bd15f1..aa23cb2601e860c3707fb608b9610c19c26ff54c 100644 (file)
--- a/gtk2_ardour/automation_controller.cc
+++ b/gtk2_ardour/automation_controller.cc
@@ -1,5 +1,5 @@
 /*
-    Copyright (C) 2007 Paul Davis 
+    Copyright (C) 2007 Paul Davis
     Author: Dave Robillard
 
     This program is free software; you can redistribute it and/or modify
@@ -18,11 +18,17 @@
 
 */
 
-#include <pbd/error.h>
+#include <iomanip>
+
+#include "pbd/error.h"
+
 #include "ardour/automation_list.h"
 #include "ardour/automation_control.h"
 #include "ardour/event_type_map.h"
 #include "ardour/automatable.h"
+#include "ardour/panner.h"
+#include "ardour/session.h"
+
 #include "ardour_ui.h"
 #include "utils.h"
 #include "automation_controller.h"
@@ -35,7 +41,7 @@ using namespace Gtk;
 
 
 AutomationController::AutomationController(boost::shared_ptr<AutomationControl> ac, Adjustment* adj)
-       : BarController(*adj, ac)
+       : BarController (*adj, ac)
        , _ignore_change(false)
        , _controllable(ac)
        , _adjustment(adj)
@@ -43,19 +49,17 @@ AutomationController::AutomationController(boost::shared_ptr<AutomationControl>
        set_name (X_("PluginSlider")); // FIXME: get yer own name!
        set_style (BarController::LeftToRight);
        set_use_parent (true);
-       
-       label_callback = sigc::mem_fun(this, &AutomationController::update_label);
-       
-       StartGesture.connect (mem_fun(*this, &AutomationController::start_touch));
-       StopGesture.connect (mem_fun(*this, &AutomationController::end_touch));
-       
+
+       StartGesture.connect (sigc::mem_fun(*this, &AutomationController::start_touch));
+       StopGesture.connect (sigc::mem_fun(*this, &AutomationController::end_touch));
+
        _adjustment->signal_value_changed().connect (
-                       mem_fun(*this, &AutomationController::value_adjusted));
-               
+                       sigc::mem_fun(*this, &AutomationController::value_adjusted));
+
        _screen_update_connection = ARDOUR_UI::RapidScreenUpdate.connect (
-                       mem_fun (*this, &AutomationController::display_effective_value));
-       
-       ac->Changed.connect (mem_fun(*this, &AutomationController::value_changed));
+                       sigc::mem_fun (*this, &AutomationController::display_effective_value));
+
+       ac->Changed.connect (_changed_connection, invalidator (*this), boost::bind (&AutomationController::value_changed, this), gui_context());
 }
 
 AutomationController::~AutomationController()
@@ -78,16 +82,21 @@ AutomationController::create(
        return boost::shared_ptr<AutomationController>(new AutomationController(ac, adjustment));
 }
 
-void
-AutomationController::update_label(char* label, int label_len)
+std::string
+AutomationController::get_label (int&)
 {
-       if (label && label_len) {
-               // Hack to display CC rounded to int
-               if (_controllable->parameter().type() == MidiCCAutomation)
-                       snprintf(label, label_len, "%d", (int)_controllable->get_value());
-               else
-                       snprintf(label, label_len, "%.3f", _controllable->get_value());
+       std::stringstream s;
+
+       // Hack to display CC rounded to int
+       if (_controllable->parameter().type() == MidiCCAutomation) {
+               s << (int)_controllable->get_value();
+       } else if (_controllable->parameter().type() == PanAutomation) {
+               s << Panner::value_as_string (_controllable->get_value ());
+       } else {
+               s << std::fixed << std::setprecision(3) << _controllable->get_value();
        }
+
+       return s.str ();
 }
 
 void
@@ -97,9 +106,9 @@ AutomationController::display_effective_value()
        //      return;
 
        float value = _controllable->get_value();
-       
+
        if (_adjustment->get_value() != value) {
-               _ignore_change = true; 
+               _ignore_change = true;
                _adjustment->set_value (value);
                _ignore_change = false;
        }
@@ -116,36 +125,52 @@ AutomationController::value_adjusted()
 void
 AutomationController::start_touch()
 {
-       _controllable->start_touch();
+       _controllable->start_touch (_controllable->session().transport_frame());
 }
 
 void
-AutomationController::end_touch()
+AutomationController::end_touch ()
 {
-       _controllable->stop_touch();
+        if (_controllable->automation_state() == Touch) {
+
+                bool mark = false;
+                double when = 0;
+                
+                if (_controllable->session().transport_rolling()) {
+                        mark = true;
+                        when = _controllable->session().transport_frame();
+                }
+
+                _controllable->stop_touch (mark, when);
+        }
 }
 
 void
 AutomationController::automation_state_changed ()
 {
-       ENSURE_GUI_THREAD(mem_fun(*this, &AutomationController::automation_state_changed));
+       ENSURE_GUI_THREAD (*this, &AutomationController::automation_state_changed)
 
        bool x = (_controllable->automation_state() != Off);
-       
+
        /* start watching automation so that things move */
-       
+
        _screen_update_connection.disconnect();
 
        if (x) {
                _screen_update_connection = ARDOUR_UI::RapidScreenUpdate.connect (
-                               mem_fun (*this, &AutomationController::display_effective_value));
+                               sigc::mem_fun (*this, &AutomationController::display_effective_value));
        }
 }
 
 void
 AutomationController::value_changed ()
 {
-       Gtkmm2ext::UI::instance()->call_slot (
-                       mem_fun(*this, &AutomationController::display_effective_value));
+       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 ()
+{
+       _screen_update_connection.disconnect ();
+}