From ddd162bdb050558f93222800ebd57d737f2c9e0d Mon Sep 17 00:00:00 2001
From: Carl Hetherington <carl@carlh.net>
Date: Tue, 24 Jan 2012 17:04:15 +0000
Subject: Make time stretch respect edit groups (#4615).

git-svn-id: svn://localhost/ardour2/branches/3.0@11334 d708f5d6-7413-0410-9779-e7cbd77b26cf
---
 gtk2_ardour/editor_drag.cc    |  4 +++-
 gtk2_ardour/editor_timefx.cc  | 15 +++++++--------
 gtk2_ardour/time_fx_dialog.cc |  7 +++----
 gtk2_ardour/time_fx_dialog.h  | 10 ++++++++--
 4 files changed, 21 insertions(+), 15 deletions(-)

(limited to 'gtk2_ardour')

diff --git a/gtk2_ardour/editor_drag.cc b/gtk2_ardour/editor_drag.cc
index 5201d3346d..dd4249f597 100644
--- a/gtk2_ardour/editor_drag.cc
+++ b/gtk2_ardour/editor_drag.cc
@@ -3292,7 +3292,9 @@ TimeFXDrag::finished (GdkEvent* /*event*/, bool movement_occurred)
 	RegionSelection rs;
 	rs.add (_primary);
 
-	if (_editor->time_stretch (rs, percentage) == -1) {
+	RegionSelection all = _editor->get_equivalent_regions (rs, ARDOUR::Properties::edit.property_id);
+
+	if (_editor->time_stretch (all, percentage) == -1) {
 		error << _("An error occurred while executing time stretch operation") << endmsg;
 	}
 }
diff --git a/gtk2_ardour/editor_timefx.cc b/gtk2_ardour/editor_timefx.cc
index 1c4162eda2..830d8999a4 100644
--- a/gtk2_ardour/editor_timefx.cc
+++ b/gtk2_ardour/editor_timefx.cc
@@ -107,8 +107,6 @@ Editor::time_fx (RegionSelection& regions, float val, bool pitching)
 
 	current_timefx = new TimeFXDialog (*this, pitching);
 
-	current_timefx->progress_bar.set_fraction (0.0f);
-
 	switch (current_timefx->run ()) {
 	case RESPONSE_ACCEPT:
 		break;
@@ -234,7 +232,6 @@ Editor::time_fx (RegionSelection& regions, float val, bool pitching)
 	current_timefx->request.quick_seek = current_timefx->quick_button.get_active();
 	current_timefx->request.antialias = !current_timefx->antialias_button.get_active();
 #endif
-	current_timefx->request.progress = 0.0f;
 	current_timefx->request.done = false;
 	current_timefx->request.cancel = false;
 
@@ -256,14 +253,10 @@ Editor::time_fx (RegionSelection& regions, float val, bool pitching)
 
 	pthread_detach (current_timefx->request.thread);
 
-	sigc::connection c = Glib::signal_timeout().connect (sigc::mem_fun (current_timefx, &TimeFXDialog::update_progress), 100);
-
 	while (!current_timefx->request.done && !current_timefx->request.cancel) {
 		gtk_main_iteration ();
 	}
 
-	c.disconnect ();
-
 	current_timefx->hide ();
 	return current_timefx->status;
 }
@@ -276,6 +269,8 @@ Editor::do_timefx (TimeFXDialog& dialog)
 	boost::shared_ptr<Region>   new_region;
 	bool in_command = false;
 
+	uint32_t const N = dialog.regions.size ();
+
 	for (RegionSelection::iterator i = dialog.regions.begin(); i != dialog.regions.end(); ) {
 		AudioRegionView* arv = dynamic_cast<AudioRegionView*>(*i);
 
@@ -324,7 +319,9 @@ Editor::do_timefx (TimeFXDialog& dialog)
 #endif
 		}
 
-		if (fx->run (region)) {
+		current_timefx->descend (1.0 / N);
+
+		if (fx->run (region, current_timefx)) {
 			dialog.status = -1;
 			dialog.request.done = true;
 			delete fx;
@@ -344,6 +341,8 @@ Editor::do_timefx (TimeFXDialog& dialog)
 			_session->add_command (new StatefulDiffCommand (playlist));
 		}
 
+		current_timefx->ascend ();
+
 		i = tmp;
 		delete fx;
 	}
diff --git a/gtk2_ardour/time_fx_dialog.cc b/gtk2_ardour/time_fx_dialog.cc
index de5786c982..97e661df84 100644
--- a/gtk2_ardour/time_fx_dialog.cc
+++ b/gtk2_ardour/time_fx_dialog.cc
@@ -186,11 +186,10 @@ TimeFXDialog::TimeFXDialog (Editor& e, bool pitch)
 	show_all_children ();
 }
 
-gint
-TimeFXDialog::update_progress ()
+void
+TimeFXDialog::update_progress_gui (float p)
 {
-	progress_bar.set_fraction (request.progress);
-	return !request.done;
+	progress_bar.set_fraction (p);
 }
 
 void
diff --git a/gtk2_ardour/time_fx_dialog.h b/gtk2_ardour/time_fx_dialog.h
index aee9d40b16..20b0538a7f 100644
--- a/gtk2_ardour/time_fx_dialog.h
+++ b/gtk2_ardour/time_fx_dialog.h
@@ -24,10 +24,13 @@
 
 #include "ardour_dialog.h"
 #include "region_selection.h"
+#include "progress_reporter.h"
 
 class Editor;
 
-struct TimeFXDialog : public ArdourDialog {
+class TimeFXDialog : public ArdourDialog, public ProgressReporter
+{
+public:
     ARDOUR::TimeFXRequest request;
     Editor&               editor;
     bool                  pitching;
@@ -58,11 +61,14 @@ struct TimeFXDialog : public ArdourDialog {
 
     TimeFXDialog (Editor& e, bool for_pitch);
 
-    gint update_progress ();
     sigc::connection first_cancel;
     sigc::connection first_delete;
     void cancel_in_progress ();
     gint delete_in_progress (GdkEventAny*);
+
+private:
+	
+    void update_progress_gui (float);
 };
 
 #endif /* __ardour_time_fx_dialog_h__ */
-- 
cgit v1.2.3