fix stupid thinko in varispeed limitation for record enable

[ardour.git] / gtk2_ardour / automation_controller.cc

diff --git a/gtk2_ardour/automation_controller.cc b/gtk2_ardour/automation_controller.cc
index 540612c8d456d49d2f3385c18a83fbe87fb624e4..79aec89f9c3bfbfe93c9dadbad3fdb46351b1d62 100644 (file)
--- a/gtk2_ardour/automation_controller.cc
+++ b/gtk2_ardour/automation_controller.cc
@@ -1,6 +1,6 @@
 /*
     Copyright (C) 2007 Paul Davis
-    Author: Dave Robillard
+    Author: David Robillard
 
     This program is free software; you can redistribute it and/or modify
     it under the terms of the GNU General Public License as published by
@@ -41,13 +41,15 @@
 using namespace ARDOUR;
 using namespace Gtk;
 
-AutomationController::AutomationController(boost::shared_ptr<Automatable> owner, boost::shared_ptr<AutomationControl> ac, Adjustment* adj)
+AutomationController::AutomationController(boost::shared_ptr<Automatable> printer, boost::shared_ptr<AutomationControl> ac, Adjustment* adj)
        : BarController (*adj, ac)
        , _ignore_change(false)
-        , _owner (owner)
+        , _printer (printer)
        , _controllable(ac)
        , _adjustment(adj)
 {
+        assert (_printer);
+
        set_name (X_("PluginSlider")); // FIXME: get yer own name!
        set_style (BarController::LeftToRight);
        set_use_parent (true);
@@ -70,43 +72,40 @@ AutomationController::~AutomationController()
 
 boost::shared_ptr<AutomationController>
 AutomationController::create(
-               boost::shared_ptr<Automatable> parent,
+               boost::shared_ptr<Automatable> printer,
                const Evoral::Parameter& param,
                boost::shared_ptr<AutomationControl> ac)
 {
-       Gtk::Adjustment* adjustment = manage(new Gtk::Adjustment(param.normal(), param.min(), param.max()));
-       if (!ac) {
-               PBD::warning << "Creating AutomationController for " << EventTypeMap::instance().to_symbol(param) << endmsg;
-               ac = boost::dynamic_pointer_cast<AutomationControl>(parent->control_factory(param));
-       } else {
-               assert(ac->parameter() == param);
-       }
-       return boost::shared_ptr<AutomationController>(new AutomationController(parent, ac, adjustment));
+       Gtk::Adjustment* adjustment = manage (new Gtk::Adjustment (param.normal(), param.min(), param.max()));
+        assert (ac);
+        assert(ac->parameter() == param);
+       return boost::shared_ptr<AutomationController>(new AutomationController(printer, ac, adjustment));
 }
 
 std::string
-AutomationController::get_label (int&)
+AutomationController::get_label (double& xpos)
 {
-        return _owner->value_as_string (_controllable);
+        xpos = 0.5;
+        return _printer->value_as_string (_controllable);
 }
 
 void
 AutomationController::display_effective_value()
 {
-       float value = _controllable->get_value();
+       double const ui_value = _controllable->user_to_ui (_controllable->get_value());
 
-       if (_adjustment->get_value() != value) {
+       if (_adjustment->get_value() != ui_value) {
                _ignore_change = true;
-               _adjustment->set_value (value);
+               _adjustment->set_value (ui_value);
                _ignore_change = false;
        }
 }
 
 void
-AutomationController::value_adjusted()
+AutomationController::value_adjusted ()
 {
        if (!_ignore_change) {
-               _controllable->set_value(_adjustment->get_value());
+               _controllable->set_value (_controllable->ui_to_user (_adjustment->get_value()));
        }
 }
 
@@ -123,7 +122,7 @@ AutomationController::end_touch ()
 
                bool mark = false;
                double when = 0;
-                
+
                if (_controllable->session().transport_rolling()) {
                        mark = true;
                        when = _controllable->session().transport_frame();