X-Git-Url: https://git.carlh.net/gitweb/?a=blobdiff_plain;f=libs%2Fardour%2Freverse.cc;h=ed3970bdfce7d454e9d96edee352ec6c216363bf;hb=e0aaed6d65f160c328cb8b56d7c6552ee15d65e2;hp=13dd531bac5dafc1eb0c427020deb1e852cf1b9d;hpb=7ff370e79895d7eb293e7214689b791bd98415fb;p=ardour.git

diff --git a/libs/ardour/reverse.cc b/libs/ardour/reverse.cc
index 13dd531bac..ed3970bdfc 100644
--- a/libs/ardour/reverse.cc
+++ b/libs/ardour/reverse.cc
@@ -15,18 +15,17 @@
     along with this program; if not, write to the Free Software
     Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
 
-    $Id$
 */
 
 #include <algorithm>
 
-#include <pbd/basename.h>
+#include "pbd/basename.h"
 
-#include <ardour/types.h>
-#include <ardour/reverse.h>
-#include <ardour/audiofilesource.h>
-#include <ardour/session.h>
-#include <ardour/audioregion.h>
+#include "ardour/types.h"
+#include "ardour/reverse.h"
+#include "ardour/audiofilesource.h"
+#include "ardour/session.h"
+#include "ardour/audioregion.h"
 
 #include "i18n.h"
 
@@ -34,7 +33,7 @@ using namespace std;
 using namespace ARDOUR;
 
 Reverse::Reverse (Session& s)
-	: AudioFilter (s)
+	: Filter (s)
 {
 }
 
@@ -43,72 +42,72 @@ Reverse::~Reverse ()
 }
 
 int
-Reverse::run (AudioRegion& region)
+Reverse::run (boost::shared_ptr<Region> r)
 {
-	AudioRegion::SourceList nsrcs;
-	AudioRegion::SourceList::iterator si;
-	const jack_nframes_t blocksize = 256 * 1048;
-	Sample buf[blocksize];
-	char * workbuf = 0;;
-	jack_nframes_t fpos;
-	jack_nframes_t fend;
-	jack_nframes_t fstart;
-	jack_nframes_t to_read;
+	SourceList nsrcs;
+	SourceList::iterator si;
+	nframes_t blocksize = 256 * 1024;
+	Sample* buf = 0;
+	nframes_t fpos;
+	nframes_t fstart;
+	nframes_t to_read;
 	int ret = -1;
 
+	boost::shared_ptr<AudioRegion> region = boost::dynamic_pointer_cast<AudioRegion>(r);
+	if (!region)
+		return ret;
+
 	/* create new sources */
 
 	if (make_new_sources (region, nsrcs)) {
 		goto out;
 	}
 
-	workbuf = new char[blocksize * 4];
-	
-	fend = region.start() + region.length();
-	fstart = region.start();
+	fstart = region->start();
 
-	if (blocksize < fend) {
-		fpos =max(fstart, fend - blocksize);
-	} else {
-		fpos = fstart;
+	if (blocksize > region->length()) {
+		blocksize = region->length();
 	}
 
-	to_read = min (region.length(), blocksize);
+	fpos = max (fstart, (fstart + region->length() - blocksize));
+
+	buf = new Sample[blocksize];
+	to_read = blocksize;
 
 	/* now read it backwards */
 
-	while (1) {
+	while (to_read) {
 
 		uint32_t n;
 
-		for (n = 0, si = nsrcs.begin(); n < region.n_channels(); ++n, ++si) {
+		for (n = 0, si = nsrcs.begin(); n < region->n_channels(); ++n, ++si) {
 
-			/* read it in */
+			/* read it in directly from the source */
 			
-			if (region.source (n).read (buf, fpos, to_read, workbuf) != to_read) {
+			if (region->audio_source (n)->read (buf, fpos, to_read) != to_read) {
 				goto out;
 			}
-			
+
 			/* swap memory order */
 			
-			for (jack_nframes_t i = 0; i < to_read/2; ++i) {
+			for (nframes_t i = 0; i < to_read/2; ++i) {
 				swap (buf[i],buf[to_read-1-i]);
 			}
-			
+
 			/* write it out */
 
-			if ((*si)->write (buf, to_read, workbuf) != to_read) {
+			boost::shared_ptr<AudioSource> asrc(boost::dynamic_pointer_cast<AudioSource>(*si));
+
+			if (asrc && asrc->write (buf, to_read) != to_read) {
 				goto out;
 			}
 		}
 
-		if (fpos == fstart) {
-			break;
-		} else if (fpos > fstart + to_read) {
+		if (fpos > fstart + blocksize) {
 			fpos -= to_read;
-			to_read = min (fstart - fpos, blocksize);
+			to_read = blocksize;
 		} else {
-			to_read = fpos-fstart;
+			to_read = fpos - fstart;
 			fpos = fstart;
 		}
 	};
@@ -117,15 +116,14 @@ Reverse::run (AudioRegion& region)
 
   out:
 
+	delete [] buf;
+
 	if (ret) {
 		for (si = nsrcs.begin(); si != nsrcs.end(); ++si) {
-			(*si)->mark_for_remove ();
-			delete *si;
+			boost::shared_ptr<AudioSource> asrc(boost::dynamic_pointer_cast<AudioSource>(*si));
+			asrc->mark_for_remove ();
 		}
 	}
-	if (workbuf) {
-		delete [] workbuf;
-	}
 	
 	return ret;
 }