From fd33fa896f787470d3c12708e02f9c7af63259c8 Mon Sep 17 00:00:00 2001
From: David Robillard <d@drobilla.net>
Date: Tue, 31 May 2011 02:59:48 +0000
Subject: Quantize notes to session tempo time, not relative to start of region
 (fix issue #4069).

git-svn-id: svn://localhost/ardour2/branches/3.0@9640 d708f5d6-7413-0410-9779-e7cbd77b26cf
---
 libs/ardour/ardour/midi_operator.h |  4 +++-
 libs/ardour/ardour/quantize.h      |  4 +++-
 libs/ardour/quantize.cc            | 18 +++++++++++++-----
 3 files changed, 19 insertions(+), 7 deletions(-)

(limited to 'libs')

diff --git a/libs/ardour/ardour/midi_operator.h b/libs/ardour/ardour/midi_operator.h
index 31e412bafe..c5def76384 100644
--- a/libs/ardour/ardour/midi_operator.h
+++ b/libs/ardour/ardour/midi_operator.h
@@ -37,7 +37,9 @@ class MidiOperator {
 	MidiOperator () {}
 	virtual ~MidiOperator() {}
 
-	virtual Command* operator() (boost::shared_ptr<ARDOUR::MidiModel>, std::vector<Evoral::Sequence<Evoral::MusicalTime>::Notes>&) = 0;
+	virtual Command* operator() (boost::shared_ptr<ARDOUR::MidiModel>,
+	                             double,
+	                             std::vector<Evoral::Sequence<Evoral::MusicalTime>::Notes>&) = 0;
 	virtual std::string name() const = 0;
 };
 
diff --git a/libs/ardour/ardour/quantize.h b/libs/ardour/ardour/quantize.h
index bf046e6c50..e1ca2b2395 100644
--- a/libs/ardour/ardour/quantize.h
+++ b/libs/ardour/ardour/quantize.h
@@ -36,7 +36,9 @@ public:
 			float strength, float swing, float threshold);
 	~Quantize ();
 
-	Command* operator() (boost::shared_ptr<ARDOUR::MidiModel>, std::vector<Evoral::Sequence<Evoral::MusicalTime>::Notes>&);
+	Command* operator() (boost::shared_ptr<ARDOUR::MidiModel>,
+	                     double position,
+	                     std::vector<Evoral::Sequence<Evoral::MusicalTime>::Notes>&);
 	std::string name() const { return std::string ("quantize"); }
 
 private:
diff --git a/libs/ardour/quantize.cc b/libs/ardour/quantize.cc
index db2cbb1d21..7ede60b578 100644
--- a/libs/ardour/quantize.cc
+++ b/libs/ardour/quantize.cc
@@ -60,8 +60,17 @@ Quantize::~Quantize ()
 }
 
 Command*
-Quantize::operator () (boost::shared_ptr<MidiModel> model, std::vector<Evoral::Sequence<Evoral::MusicalTime>::Notes>& seqs)
+Quantize::operator () (boost::shared_ptr<MidiModel> model,
+                       double position,
+                       std::vector<Evoral::Sequence<Evoral::MusicalTime>::Notes>& seqs)
 {
+	/* Calculate offset from start of model to next closest quantize step,
+	   to quantize relative to actual session beats (etc.) rather than from the
+	   start of the model.
+	*/
+	const double round_pos = ceil(position / _start_grid) * _start_grid;
+	const double offset    = round_pos - position;
+
 	bool even;
 	MidiModel::NoteDiffCommand* cmd = new MidiModel::NoteDiffCommand (model, "quantize");
 
@@ -71,9 +80,8 @@ Quantize::operator () (boost::shared_ptr<MidiModel> model, std::vector<Evoral::S
 
 		for (Evoral::Sequence<MidiModel::TimeType>::Notes::iterator i = (*s).begin(); i != (*s).end(); ++i) {
 
-			double new_start = round ((*i)->time() / _start_grid) * _start_grid;
-			double new_end = round ((*i)->end_time() / _end_grid) * _end_grid;
-			double delta;
+			double new_start = round ((*i)->time() / _start_grid) * _start_grid + offset;
+			double new_end = round ((*i)->end_time() / _end_grid) * _end_grid + offset;
 
 			if (_swing > 0.0 && !even) {
 
@@ -97,7 +105,7 @@ Quantize::operator () (boost::shared_ptr<MidiModel> model, std::vector<Evoral::S
 
 			}
 
-			delta = new_start - (*i)->time();
+			double delta = new_start - (*i)->time();
 
 			if (fabs (delta) >= _threshold) {
 				if (_snap_start) {
-- 
cgit v1.2.3