X-Git-Url: https://git.carlh.net/gitweb/?a=blobdiff_plain;f=gtk2_ardour%2Fautomation_controller.cc;h=a80a4742327cf41eadd782287b83b0af455c9af9;hb=e0b2764b912b15ddb329833974e0bef037969d37;hp=3af5f3444511d277810f62c9d74c3d84d4b8b512;hpb=8b68ed57ec5c075261639435be3290c3ba8177eb;p=ardour.git

diff --git a/gtk2_ardour/automation_controller.cc b/gtk2_ardour/automation_controller.cc
index 3af5f34445..a80a474232 100644
--- 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
@@ -19,14 +19,12 @@
 */
 
 #include <iomanip>
+#include <cmath>
 
 #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/automation_control.h"
 #include "ardour/session.h"
 
 #include "ardour_ui.h"
@@ -39,13 +37,15 @@
 using namespace ARDOUR;
 using namespace Gtk;
 
-
-AutomationController::AutomationController(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)
+        , _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);
@@ -68,57 +68,49 @@ 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(ac, adjustment));
+	Gtk::Adjustment* adjustment = manage (
+		new Gtk::Adjustment (
+			ac->internal_to_interface (param.normal()),
+			ac->internal_to_interface (param.min()),
+			ac->internal_to_interface (param.max()),
+			(param.max() - param.min()) / 100.0,
+			(param.max() - param.min()) / 10.0
+			)
+		);
+
+        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)
 {
-	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 ();
+        xpos = 0.5;
+        return _printer->value_as_string (_controllable);
 }
 
 void
 AutomationController::display_effective_value()
 {
-	//if ( ! _controllable->list()->automation_playback())
-	//	return;
+	double const interface_value = _controllable->internal_to_interface (_controllable->get_value());
 
-	float value = _controllable->get_value();
-
-	if (_adjustment->get_value() != value) {
+	if (_adjustment->get_value () != interface_value) {
 		_ignore_change = true;
-		_adjustment->set_value (value);
+		_adjustment->set_value (interface_value);
 		_ignore_change = false;
 	}
 }
 
 void
-AutomationController::value_adjusted()
+AutomationController::value_adjusted ()
 {
 	if (!_ignore_change) {
-		_controllable->set_value(_adjustment->get_value());
+		_controllable->set_value (_controllable->interface_to_internal (_adjustment->get_value()));
 	}
 }
 
@@ -135,7 +127,7 @@ AutomationController::end_touch ()
 
 		bool mark = false;
 		double when = 0;
-                
+
 		if (_controllable->session().transport_rolling()) {
 			mark = true;
 			when = _controllable->session().transport_frame();
@@ -145,23 +137,6 @@ AutomationController::end_touch ()
 	}
 }
 
-void
-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 (
-				sigc::mem_fun (*this, &AutomationController::display_effective_value));
-	}
-}
-
 void
 AutomationController::value_changed ()
 {