clean up debugging

[ardour.git] / libs / ardour / auditioner.cc

diff --git a/libs/ardour/auditioner.cc b/libs/ardour/auditioner.cc
index f955c75ea4877c3afcef51708d58a232561ac513..d7605f26e1b3bc4d28da7dae431dc8f54c9207ef 100644 (file)
--- a/libs/ardour/auditioner.cc
+++ b/libs/ardour/auditioner.cc
@@ -42,6 +42,10 @@ using namespace PBD;
 
 Auditioner::Auditioner (Session& s)
        : AudioTrack (s, "auditioner", Route::Hidden)
+        , current_frame (0)
+        , _auditioning (0)
+        , length (0)
+        , via_monitor (false)
 {
 }
 
@@ -55,19 +59,28 @@ Auditioner::init ()
        string left = _session.config.get_auditioner_output_left();
        string right = _session.config.get_auditioner_output_right();
 
+       vector<string> outputs;
+       _session.engine().get_physical_outputs (DataType::AUDIO, outputs);
+
        if (left == "default") {
                 if (_session.monitor_out()) {
                         left = _session.monitor_out()->input()->audio (0)->name();
+                        via_monitor = true;
                 } else {
-                        left = _session.engine().get_nth_physical_output (DataType::AUDIO, 0);
+                       if (outputs.size() > 0) {
+                               left = outputs[0];
+                       }
                 }
        }
 
        if (right == "default") {
                 if (_session.monitor_out()) {
                         right = _session.monitor_out()->input()->audio (1)->name();
+                        via_monitor = true;
                 } else {
-                        right = _session.engine().get_nth_physical_output (DataType::AUDIO, 1);
+                       if (outputs.size() > 1) {
+                               right = outputs[1];
+                       }
                 }
        }
 
@@ -91,9 +104,6 @@ Auditioner::init ()
 
        _output->changed.connect_same_thread (*this, boost::bind (&Auditioner::output_changed, this, _1, _2));
 
-       the_region.reset ((AudioRegion*) 0);
-       g_atomic_int_set (&_auditioning, 0);
-
         return 0;
 }
 
@@ -124,7 +134,7 @@ Auditioner::audition_current_playlist ()
 
        Glib::Mutex::Lock lm (lock);
        _diskstream->seek (0);
-       length = _diskstream->playlist()->get_maximum_extent();
+       length = _diskstream->playlist()->get_extent().second;
        current_frame = 0;
 
        /* force a panner reset now that we have all channels */
@@ -166,11 +176,15 @@ Auditioner::audition_region (boost::shared_ptr<Region> region)
        }
 
         ProcessorStreams ps;
-        if (configure_processors (&ps)) {
-                error << string_compose (_("Cannot setup auditioner processing flow for %1 channels"), 
-                                         _diskstream->n_channels()) << endmsg;
-                return;
-        }
+       {
+               Glib::Mutex::Lock lm (AudioEngine::instance()->process_lock ());
+
+               if (configure_processors (&ps)) {
+                       error << string_compose (_("Cannot setup auditioner processing flow for %1 channels"), 
+                                                _diskstream->n_channels()) << endmsg;
+                       return;
+               }
+       }
 
        /* force a panner reset now that we have all channels */
 
@@ -179,7 +193,7 @@ Auditioner::audition_region (boost::shared_ptr<Region> region)
        length = the_region->length();
 
        int dir;
-       nframes_t offset = the_region->sync_offset (dir);
+       framecnt_t offset = the_region->sync_offset (dir);
 
        /* can't audition from a negative sync point */
 
@@ -194,10 +208,10 @@ Auditioner::audition_region (boost::shared_ptr<Region> region)
 }
 
 int
-Auditioner::play_audition (nframes_t nframes)
+Auditioner::play_audition (framecnt_t nframes)
 {
-       bool need_butler;
-       nframes_t this_nframes;
+       bool need_butler = false;
+       framecnt_t this_nframes;
        int ret;
 
        if (g_atomic_int_get (&_auditioning) == 0) {
@@ -207,14 +221,11 @@ Auditioner::play_audition (nframes_t nframes)
 
        this_nframes = min (nframes, length - current_frame);
 
-       _diskstream->prepare ();
-
-       if ((ret = roll (this_nframes, current_frame, current_frame + nframes, false, false, false)) != 0) {
+       if ((ret = roll (this_nframes, current_frame, current_frame + nframes, false, false, false, need_butler)) != 0) {
                silence (nframes);
                return ret;
        }
 
-       need_butler = _diskstream->commit (this_nframes);
        current_frame += this_nframes;
 
        if (current_frame >= length) {
@@ -228,11 +239,15 @@ Auditioner::play_audition (nframes_t nframes)
 void
 Auditioner::output_changed (IOChange change, void* /*src*/)
 {
-       if (change & ConnectionsChanged) {
+       if (change.type & IOChange::ConnectionsChanged) {
                string phys;
                vector<string> connections;
+               vector<string> outputs;
+               _session.engine().get_physical_outputs (DataType::AUDIO, outputs);
                if (_output->nth (0)->get_connections (connections)) {
-                       phys = _session.engine().get_nth_physical_output (DataType::AUDIO, 0);
+                       if (outputs.size() > 0) {
+                               phys = outputs[0];
+                       }
                        if (phys != connections[0]) {
                                _session.config.set_auditioner_output_left (connections[0]);
                        } else {
@@ -245,7 +260,9 @@ Auditioner::output_changed (IOChange change, void* /*src*/)
                connections.clear ();
 
                if (_output->nth (1)->get_connections (connections)) {
-                       phys = _session.engine().get_nth_physical_output (DataType::AUDIO, 1);
+                       if (outputs.size() > 1) {
+                               phys = outputs[1];
+                       }
                        if (phys != connections[0]) {
                                _session.config.set_auditioner_output_right (connections[0]);
                        } else {