Add a null check to ports when loading an export channel from xml + fix indentation...
[ardour.git] / libs / ardour / audioengine.cc
index 79aa5d9ff8762ca0cd4fc0130f799a1739ea78fe..7da687657759603ce29b1a5e6916d33be683c081 100644 (file)
@@ -32,7 +32,9 @@
 #include "pbd/stacktrace.h"
 #include "pbd/unknown_type.h"
 
-#include "midi++/jack.h"
+#include "midi++/port.h"
+#include "midi++/mmc.h"
+#include "midi++/manager.h"
 
 #include "ardour/amp.h"
 #include "ardour/audio_port.h"
@@ -74,7 +76,7 @@ AudioEngine::AudioEngine (string client_name, string session_uuid)
        _running = false;
        _has_run = false;
        last_monitor_check = 0;
-       monitor_check_interval = max_frames;
+       monitor_check_interval = INT32_MAX;
        _processed_frames = 0;
        _usecs_per_cycle = 0;
        _jack = 0;
@@ -145,7 +147,7 @@ _thread_init_callback (void * /*arg*/)
 
        SessionEvent::create_per_thread_pool (X_("Audioengine"), 512);
 
-       MIDI::JACK_MidiPort::set_process_thread (pthread_self());
+       MIDI::Port::set_process_thread (pthread_self());
 }
 
 typedef void (*_JackInfoShutdownCallback)(jack_status_t code, const char* reason, void *arg);
@@ -174,7 +176,7 @@ AudioEngine::start ()
 
        if (!_running) {
 
-               nframes_t blocksize = jack_get_buffer_size (_priv_jack);
+               pframes_t blocksize = jack_get_buffer_size (_priv_jack);
 
                if (_session) {
                        BootMessage (_("Connect session to engine"));
@@ -241,7 +243,6 @@ AudioEngine::start ()
                         error << _("Cannot create temporary MIDI port to determine MIDI buffer size") << endmsg;
                 } else {
                         _raw_buffer_sizes[DataType::MIDI] = jack_midi_max_event_size (jack_port_get_buffer(midi_port, blocksize));
-                        cerr << "MIDI port buffers = " << _raw_buffer_sizes[DataType::MIDI] << endl;
                         jack_port_unregister (_priv_jack, midi_port);
                 }
        }
@@ -260,16 +261,20 @@ AudioEngine::stop (bool forever)
                } else {
                        jack_deactivate (_priv_jack);
                        Stopped(); /* EMIT SIGNAL */
-                       MIDI::JACK_MidiPort::JackHalted (); /* EMIT SIGNAL */
+                       MIDI::Port::JackHalted (); /* EMIT SIGNAL */
                }
        }
 
+        if (forever) {
+                stop_metering_thread ();
+        }
+
        return _running ? -1 : 0;
 }
 
 
 bool
-AudioEngine::get_sync_offset (nframes_t& offset) const
+AudioEngine::get_sync_offset (pframes_t& offset) const
 {
 
 #ifdef HAVE_JACK_VIDEO_SUPPORT
@@ -295,14 +300,14 @@ AudioEngine::get_sync_offset (nframes_t& offset) const
 }
 
 void
-AudioEngine::_jack_timebase_callback (jack_transport_state_t state, nframes_t nframes,
+AudioEngine::_jack_timebase_callback (jack_transport_state_t state, pframes_t nframes,
                                      jack_position_t* pos, int new_position, void *arg)
 {
        static_cast<AudioEngine*> (arg)->jack_timebase_callback (state, nframes, pos, new_position);
 }
 
 void
-AudioEngine::jack_timebase_callback (jack_transport_state_t state, nframes_t nframes,
+AudioEngine::jack_timebase_callback (jack_transport_state_t state, pframes_t nframes,
                                     jack_position_t* pos, int new_position)
 {
        if (_jack && _session && _session->synced_to_jack()) {
@@ -363,7 +368,7 @@ AudioEngine::_graph_order_callback (void *arg)
  * @param arg User argument passed by JACK, which will be the AudioEngine*.
  */
 int
-AudioEngine::_process_callback (nframes_t nframes, void *arg)
+AudioEngine::_process_callback (pframes_t nframes, void *arg)
 {
        return static_cast<AudioEngine *> (arg)->process_callback (nframes);
 }
@@ -395,11 +400,11 @@ AudioEngine::_connect_callback (jack_port_id_t /*id_a*/, jack_port_id_t /*id_b*/
 }
 
 void
-AudioEngine::split_cycle (nframes_t offset)
+AudioEngine::split_cycle (pframes_t offset)
 {
        /* caller must hold process lock */
 
-       Port::increment_port_offset (offset);
+       AudioPort::increment_port_offset (offset);
 
        /* tell all Ports that we're going to start a new (split) cycle */
 
@@ -411,7 +416,7 @@ AudioEngine::split_cycle (nframes_t offset)
 }
 
 void
-AudioEngine::finish_process_cycle (int status)
+AudioEngine::finish_process_cycle (int /* status*/ )
 {
         GET_PRIVATE_JACK_POINTER(_jack);
         jack_cycle_signal (_jack, 0);
@@ -429,7 +434,7 @@ AudioEngine::process_thread ()
 
         while (1) {
                 GET_PRIVATE_JACK_POINTER_RET(_jack,0);
-                jack_nframes_t nframes = jack_cycle_wait (_jack);
+                pframes_t nframes = jack_cycle_wait (_jack);
 
                 if (process_callback (nframes)) {
                         cerr << "--- process\n";
@@ -447,19 +452,19 @@ AudioEngine::process_thread ()
  * @param nframes Number of frames to process.
  */
 int
-AudioEngine::process_callback (nframes_t nframes)
+AudioEngine::process_callback (pframes_t nframes)
 {
        GET_PRIVATE_JACK_POINTER_RET(_jack,0);
        // CycleTimer ct ("AudioEngine::process");
        Glib::Mutex::Lock tm (_process_lock, Glib::TRY_LOCK);
 
        /// The number of frames that will have been processed when we've finished
-       nframes_t next_processed_frames;
+       pframes_t next_processed_frames;
 
        /* handle wrap around of total frames counter */
 
-       if (max_frames - _processed_frames < nframes) {
-               next_processed_frames = nframes - (max_frames - _processed_frames);
+       if (max_framepos - _processed_frames < nframes) {
+               next_processed_frames = nframes - (max_framepos - _processed_frames);
        } else {
                next_processed_frames = _processed_frames + nframes;
        }
@@ -482,7 +487,7 @@ AudioEngine::process_callback (nframes_t nframes)
        /* tell all relevant objects that we're starting a new cycle */
 
        Delivery::CycleStart (nframes);
-       Port::set_port_offset (0);
+       AudioPort::set_port_offset (0);
        InternalReturn::CycleStart (nframes);
 
        /* tell all Ports that we're starting a new cycle */
@@ -493,7 +498,11 @@ AudioEngine::process_callback (nframes_t nframes)
                (*i)->cycle_start (nframes);
        }
 
-       if (_freewheeling) {
+       /* test if we are freewheeling and there are freewheel signals connected.
+           ardour should act normally even when freewheeling unless /it/ is exporting */
+
+
+       if (_freewheeling && !Freewheel.empty()) {
                /* emit the Freewheel signal and stop freewheeling in the event of trouble 
                 */
                 boost::optional<int> r = Freewheel (nframes);
@@ -562,13 +571,13 @@ AudioEngine::process_callback (nframes_t nframes)
 }
 
 int
-AudioEngine::_sample_rate_callback (nframes_t nframes, void *arg)
+AudioEngine::_sample_rate_callback (pframes_t nframes, void *arg)
 {
        return static_cast<AudioEngine *> (arg)->jack_sample_rate_callback (nframes);
 }
 
 int
-AudioEngine::jack_sample_rate_callback (nframes_t nframes)
+AudioEngine::jack_sample_rate_callback (pframes_t nframes)
 {
        _frame_rate = nframes;
        _usecs_per_cycle = (int) floor ((((double) frames_per_cycle() / nframes)) * 1000000.0);
@@ -588,13 +597,13 @@ AudioEngine::jack_sample_rate_callback (nframes_t nframes)
 }
 
 int
-AudioEngine::_bufsize_callback (nframes_t nframes, void *arg)
+AudioEngine::_bufsize_callback (pframes_t nframes, void *arg)
 {
        return static_cast<AudioEngine *> (arg)->jack_bufsize_callback (nframes);
 }
 
 int
-AudioEngine::jack_bufsize_callback (nframes_t nframes)
+AudioEngine::jack_bufsize_callback (pframes_t nframes)
 {
         bool need_midi_size = true;
         bool need_audio_size = true;
@@ -685,7 +694,7 @@ AudioEngine::set_session (Session *s)
 
                start_metering_thread ();
                
-               nframes_t blocksize = jack_get_buffer_size (_jack);
+               pframes_t blocksize = jack_get_buffer_size (_jack);
                
                /* page in as much of the session process code as we
                   can before we really start running.
@@ -838,8 +847,6 @@ AudioEngine::unregister_port (Port& port)
 int
 AudioEngine::connect (const string& source, const string& destination)
 {
-       /* caller must hold process lock */
-
        int ret;
 
        if (!_running) {
@@ -855,8 +862,8 @@ AudioEngine::connect (const string& source, const string& destination)
        string d = make_port_name_non_relative (destination);
 
 
-       Port* src = get_port_by_name_locked (s);
-       Port* dst = get_port_by_name_locked (d);
+       Port* src = get_port_by_name (s);
+       Port* dst = get_port_by_name (d);
 
        if (src) {
                ret = src->connect (d);
@@ -881,8 +888,6 @@ AudioEngine::connect (const string& source, const string& destination)
 int
 AudioEngine::disconnect (const string& source, const string& destination)
 {
-       /* caller must hold process lock */
-
        int ret;
 
        if (!_running) {
@@ -897,8 +902,8 @@ AudioEngine::disconnect (const string& source, const string& destination)
        string s = make_port_name_non_relative (source);
        string d = make_port_name_non_relative (destination);
 
-       Port* src = get_port_by_name_locked (s);
-       Port* dst = get_port_by_name_locked (d);
+       Port* src = get_port_by_name (s);
+       Port* dst = get_port_by_name (d);
 
        if (src) {
                        ret = src->disconnect (d);
@@ -928,14 +933,14 @@ AudioEngine::disconnect (Port& port)
        return port.disconnect_all ();
 }
 
-ARDOUR::nframes_t
+ARDOUR::framecnt_t
 AudioEngine::frame_rate () const
 {
-       GET_PRIVATE_JACK_POINTER_RET (_jack,0);
+       GET_PRIVATE_JACK_POINTER_RET (_jack, 0);
        if (_frame_rate == 0) {
-         return (_frame_rate = jack_get_sample_rate (_priv_jack));
+               return (_frame_rate = jack_get_sample_rate (_priv_jack));
        } else {
-         return _frame_rate;
+               return _frame_rate;
        }
 }
 
@@ -946,52 +951,38 @@ AudioEngine::raw_buffer_size (DataType t)
        return (s != _raw_buffer_sizes.end()) ? s->second : 0;
 }
 
-ARDOUR::nframes_t
+ARDOUR::pframes_t
 AudioEngine::frames_per_cycle () const
 {
        GET_PRIVATE_JACK_POINTER_RET (_jack,0);
        if (_buffer_size == 0) {
-         return (_buffer_size = jack_get_buffer_size (_jack));
+               return (_buffer_size = jack_get_buffer_size (_jack));
        } else {
-         return _buffer_size;
+               return _buffer_size;
        }
 }
 
-/** @param name Full name of port (including prefix:)
+/** @param name Full or short name of port
  *  @return Corresponding Port*, or 0.  This object remains the property of the AudioEngine
  *  so must not be deleted.
  */
 Port *
 AudioEngine::get_port_by_name (const string& portname)
 {
-       string s;
-       if (portname.find_first_of (':') == string::npos) {
-               s = make_port_name_non_relative (portname);
-       } else {
-               s = portname;
-       }
-
-       Glib::Mutex::Lock lm (_process_lock);
-       return get_port_by_name_locked (s);
-}
-
-Port *
-AudioEngine::get_port_by_name_locked (const string& portname)
-{
-       /* caller must hold process lock */
-
        if (!_running) {
                if (!_has_run) {
-                       fatal << _("get_port_by_name_locked() called before engine was started") << endmsg;
+                       fatal << _("get_port_by_name() called before engine was started") << endmsg;
                        /*NOTREACHED*/
                } else {
                        return 0;
                }
        }
 
-       if (portname.substr (0, jack_client_name.length ()) != jack_client_name) {
-               /* not an ardour: port */
-               return 0;
+       if (portname.find_first_of (':') != string::npos) {
+               if (portname.substr (0, jack_client_name.length ()) != jack_client_name) {
+                       /* not an ardour: port */
+                       return 0;
+               }
        }
 
        std::string const rel = make_port_name_relative (portname);
@@ -1071,7 +1062,7 @@ AudioEngine::halted (void *arg)
 
        if (was_running) {
                ae->Halted(""); /* EMIT SIGNAL */
-               MIDI::JACK_MidiPort::JackHalted (); /* EMIT SIGNAL */
+               MIDI::Port::JackHalted (); /* EMIT SIGNAL */
        }
 }
 
@@ -1103,58 +1094,49 @@ AudioEngine::can_request_hardware_monitoring ()
        return true;
 }
 
-
-uint32_t
-AudioEngine::n_physical_outputs (DataType type) const
+ChanCount
+AudioEngine::n_physical (unsigned long flags) const
 {
-       GET_PRIVATE_JACK_POINTER_RET (_jack,0);
-       const char ** ports;
-       uint32_t cnt = 0;
+       ChanCount c;
+       
+       GET_PRIVATE_JACK_POINTER_RET (_jack, c);
 
-       if ((ports = jack_get_ports (_priv_jack, NULL, type.to_jack_type(), JackPortIsPhysical|JackPortIsInput)) == 0) {
-               return 0;
+       const char ** ports = jack_get_ports (_priv_jack, NULL, NULL, JackPortIsPhysical | flags);
+       if (ports == 0) {
+               return c;
        }
 
        for (uint32_t i = 0; ports[i]; ++i) {
-                if (!strstr (ports[i], "Midi-Through")) {
-                        cnt++;
-                }
-        }
+               if (!strstr (ports[i], "Midi-Through")) {
+                       DataType t (jack_port_type (jack_port_by_name (_jack, ports[i])));
+                       c.set (t, c.get (t) + 1);
+               }
+       }
 
        free (ports);
 
-       return cnt;
+       return c;
 }
 
-uint32_t
-AudioEngine::n_physical_inputs (DataType type) const
+ChanCount
+AudioEngine::n_physical_inputs () const
 {
-       GET_PRIVATE_JACK_POINTER_RET (_jack,0);
-       const char ** ports;
-       uint32_t cnt = 0;
-        
-       if ((ports = jack_get_ports (_priv_jack, NULL, type.to_jack_type(), JackPortIsPhysical|JackPortIsOutput)) == 0) {
-               return 0;
-       }
-
-       for (uint32_t i = 0; ports[i]; ++i) {
-                if (!strstr (ports[i], "Midi-Through")) {
-                        cnt++;
-                }
-        }
-
-       free (ports);
+       return n_physical (JackPortIsInput);
+}
 
-       return cnt;
+ChanCount
+AudioEngine::n_physical_outputs () const
+{
+       return n_physical (JackPortIsOutput);
 }
 
 void
-AudioEngine::get_physical_inputs (DataType type, vector<string>& ins)
+AudioEngine::get_physical (DataType type, unsigned long flags, vector<string>& phy)
 {
        GET_PRIVATE_JACK_POINTER (_jack);
        const char ** ports;
 
-       if ((ports = jack_get_ports (_priv_jack, NULL, type.to_jack_type(), JackPortIsPhysical|JackPortIsOutput)) == 0) {
+       if ((ports = jack_get_ports (_priv_jack, NULL, type.to_jack_type(), JackPortIsPhysical | flags)) == 0) {
                return;
        }
 
@@ -1163,60 +1145,28 @@ AudioEngine::get_physical_inputs (DataType type, vector<string>& ins)
                         if (strstr (ports[i], "Midi-Through")) {
                                 continue;
                         }
-                       ins.push_back (ports[i]);
+                       phy.push_back (ports[i]);
                }
                free (ports);
        }
 }
 
+/** Get physical ports for which JackPortIsOutput is set; ie those that correspond to
+ *  a physical input connector.
+ */
 void
-AudioEngine::get_physical_outputs (DataType type, vector<string>& outs)
+AudioEngine::get_physical_inputs (DataType type, vector<string>& ins)
 {
-       GET_PRIVATE_JACK_POINTER (_jack);
-       const char ** ports;
-       uint32_t i = 0;
-
-       if ((ports = jack_get_ports (_priv_jack, NULL, type.to_jack_type(), JackPortIsPhysical|JackPortIsInput)) == 0) {
-               return;
-       }
-
-       for (i = 0; ports[i]; ++i) {
-                if (strstr (ports[i], "Midi-Through")) {
-                        continue;
-                }
-               outs.push_back (ports[i]);
-       }
-       free (ports);
+       get_physical (type, JackPortIsOutput, ins);
 }
 
-string
-AudioEngine::get_nth_physical (DataType type, uint32_t n, int flag)
+/** Get physical ports for which JackPortIsInput is set; ie those that correspond to
+ *  a physical output connector.
+ */
+void
+AudioEngine::get_physical_outputs (DataType type, vector<string>& outs)
 {
-       GET_PRIVATE_JACK_POINTER_RET (_jack,"");
-       const char ** ports;
-       uint32_t i;
-       uint32_t idx;
-       string ret;
-
-       assert(type != DataType::NIL);
-
-       if ((ports = jack_get_ports (_priv_jack, NULL, type.to_jack_type(), JackPortIsPhysical|flag)) == 0) {
-               return ret;
-       }
-
-       for (i = 0, idx = 0; idx < n && ports[i]; ++i) {
-                if (!strstr (ports[i], "Midi-Through")) {
-                        ++idx;
-                }
-        }
-
-       if (ports[idx]) {
-               ret = ports[idx];
-       }
-
-       free ((const char **) ports);
-
-       return ret;
+       get_physical (type, JackPortIsInput, outs);
 }
 
 void
@@ -1240,7 +1190,7 @@ AudioEngine::transport_start ()
 }
 
 void
-AudioEngine::transport_locate (nframes_t where)
+AudioEngine::transport_locate (framepos_t where)
 {
        GET_PRIVATE_JACK_POINTER (_jack);
        // cerr << "tell JACK to locate to " << where << endl;
@@ -1355,7 +1305,7 @@ AudioEngine::disconnect_from_jack ()
        if (_running) {
                _running = false;
                Stopped(); /* EMIT SIGNAL */
-               MIDI::JACK_MidiPort::JackHalted (); /* EMIT SIGNAL */
+               MIDI::Port::JackHalted (); /* EMIT SIGNAL */
        }
 
        return 0;
@@ -1393,6 +1343,8 @@ AudioEngine::reconnect_to_jack ()
 
        GET_PRIVATE_JACK_POINTER_RET (_jack,-1);
 
+       MIDI::Manager::instance()->reestablish (_priv_jack);
+       
        if (_session) {
                _session->reset_jack_connection (_priv_jack);
                 jack_bufsize_callback (jack_get_buffer_size (_priv_jack));
@@ -1433,6 +1385,8 @@ AudioEngine::reconnect_to_jack ()
                (*i)->reconnect ();
        }
 
+       MIDI::Manager::instance()->reconnect ();
+
        Running (); /* EMIT SIGNAL*/
 
        start_metering_thread ();
@@ -1441,7 +1395,7 @@ AudioEngine::reconnect_to_jack ()
 }
 
 int
-AudioEngine::request_buffer_size (nframes_t nframes)
+AudioEngine::request_buffer_size (pframes_t nframes)
 {
        GET_PRIVATE_JACK_POINTER_RET (_jack, -1);
 
@@ -1505,19 +1459,18 @@ AudioEngine::is_realtime () const
        return jack_is_realtime (_priv_jack);
 }
 
-pthread_t
-AudioEngine::create_process_thread (boost::function<void()> f, size_t stacksize)
+int
+AudioEngine::create_process_thread (boost::function<void()> f, pthread_t* thread, size_t stacksize)
 {
         GET_PRIVATE_JACK_POINTER_RET (_jack, 0);
-        pthread_t thread;
         ThreadData* td = new ThreadData (this, f, stacksize);
 
-        if (jack_client_create_thread (_priv_jack, &thread, jack_client_real_time_priority (_priv_jack), 
+        if (jack_client_create_thread (_priv_jack, thread, jack_client_real_time_priority (_priv_jack), 
                                        jack_is_realtime (_priv_jack), _start_process_thread, td)) {
                 return -1;
         } 
 
-        return thread;
+        return 0;
 }
 
 void*