PBD::canonical_path will no longer throw so change test

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

diff --git a/libs/ardour/port_manager.cc b/libs/ardour/port_manager.cc
index acbcb016d5d030b15f0ac8bc6b9ae1ac83521a1d..10c19a9c67547f4cd07aecbdec4f05185a06e50e 100644 (file)
--- a/libs/ardour/port_manager.cc
+++ b/libs/ardour/port_manager.cc
@@ -17,6 +17,7 @@
 
 */
 
+#include "pbd/convert.h"
 #include "pbd/error.h"
 
 #include "ardour/async_midi_port.h"
@@ -26,8 +27,10 @@
 #include "ardour/midi_port.h"
 #include "ardour/midiport_manager.h"
 #include "ardour/port_manager.h"
+#include "ardour/profile.h"
+#include "ardour/session.h"
 
-#include "i18n.h"
+#include "pbd/i18n.h"
 
 using namespace ARDOUR;
 using namespace PBD;
@@ -73,20 +76,14 @@ PortManager::make_port_name_relative (const string& portname) const
                return portname;
        }
 
-       string::size_type len;
-       string::size_type n;
-       string self = _backend->my_name();
-
-       len = portname.length();
+       string::size_type colon = portname.find (':');
 
-       for (n = 0; n < len; ++n) {
-               if (portname[n] == ':') {
-                       break;
-               }
+       if (colon == string::npos) {
+               return portname;
        }
 
-       if ((n != len) && (portname.substr (0, n) == self)) {
-               return portname.substr (n+1);
+       if (portname.substr (0, colon) == _backend->my_name()) {
+               return portname.substr (colon+1);
        }
 
        return portname;
@@ -162,21 +159,23 @@ void
 PortManager::get_physical_outputs (DataType type, std::vector<std::string>& s)
 {
        if (!_backend) {
+               s.clear ();
                return;
        }
        _backend->get_physical_outputs (type, s);
 }
- 
+
 void
 PortManager::get_physical_inputs (DataType type, std::vector<std::string>& s)
 {
        if (!_backend) {
+               s.clear ();
                return;
        }
 
        _backend->get_physical_inputs (type, s);
 }
- 
+
 ChanCount
 PortManager::n_physical_outputs () const
 {
@@ -186,7 +185,7 @@ PortManager::n_physical_outputs () const
 
        return _backend->n_physical_outputs ();
 }
- 
+
 ChanCount
 PortManager::n_physical_inputs () const
 {
@@ -238,7 +237,7 @@ PortManager::port_renamed (const std::string& old_relative_name, const std::stri
        RCUWriter<Ports> writer (ports);
        boost::shared_ptr<Ports> p = writer.get_copy();
        Ports::iterator x = p->find (old_relative_name);
-       
+
        if (x != p->end()) {
                boost::shared_ptr<Port> port = x->second;
                p->erase (x);
@@ -354,6 +353,12 @@ PortManager::register_output_port (DataType type, const string& portname, bool a
 int
 PortManager::unregister_port (boost::shared_ptr<Port> port)
 {
+       /* This is a little subtle. We do not call the backend's port
+        * unregistration code from here. That is left for the Port
+        * destructor. We are trying to drop references to the Port object
+        * here, so that its destructor will run and it will unregister itself.
+        */
+
        /* caller must hold process lock */
 
        {
@@ -389,6 +394,40 @@ PortManager::connected (const string& port_name)
        return _backend->connected (handle);
 }
 
+bool
+PortManager::physically_connected (const string& port_name)
+{
+       if (!_backend) {
+               return false;
+       }
+
+       PortEngine::PortHandle handle = _backend->get_port_by_name (port_name);
+
+       if (!handle) {
+               return false;
+       }
+
+       return _backend->physically_connected (handle);
+}
+
+int
+PortManager::get_connections (const string& port_name, std::vector<std::string>& s)
+{
+       if (!_backend) {
+               s.clear ();
+               return 0;
+       }
+
+       PortEngine::PortHandle handle = _backend->get_port_by_name (port_name);
+
+       if (!handle) {
+               s.clear ();
+               return 0;
+       }
+
+       return _backend->get_connections (handle, s);
+}
+
 int
 PortManager::connect (const string& source, const string& destination)
 {
@@ -491,9 +530,9 @@ PortManager::reconnect_ports ()
 
        if (!Profile->get_trx()) {
                /* re-establish connections */
-               
+
                DEBUG_TRACE (DEBUG::Ports, string_compose ("reconnect %1 ports\n", p->size()));
-               
+
                for (Ports::iterator i = p->begin(); i != p->end(); ++i) {
                        i->second->reconnect ();
                }
@@ -525,7 +564,7 @@ PortManager::connect_callback (const string& a, const string& b, bool conn)
                port_b, b,
                conn
                ); /* EMIT SIGNAL */
-}      
+}
 
 void
 PortManager::registration_callback ()
@@ -544,7 +583,7 @@ PortManager::can_request_input_monitoring () const
 
        return _backend->can_monitor_input ();
 }
- 
+
 void
 PortManager::request_input_monitoring (const string& name, bool yn) const
 {
@@ -558,7 +597,7 @@ PortManager::request_input_monitoring (const string& name, bool yn) const
                _backend->request_input_monitoring (ph, yn);
        }
 }
- 
+
 void
 PortManager::ensure_input_monitoring (const string& name, bool yn) const
 {
@@ -579,7 +618,7 @@ PortManager::port_name_size() const
        if (!_backend) {
                return 0;
        }
-       
+
        return _backend->port_name_size ();
 }
 
@@ -589,7 +628,7 @@ PortManager::my_name() const
        if (!_backend) {
                return string();
        }
-       
+
        return _backend->my_name();
 }
 
@@ -633,9 +672,18 @@ PortManager::cycle_end (pframes_t nframes)
 }
 
 void
-PortManager::silence (pframes_t nframes)
+PortManager::silence (pframes_t nframes, Session *s)
 {
        for (Ports::iterator i = _cycle_ports->begin(); i != _cycle_ports->end(); ++i) {
+               if (s && i->second == s->mtc_output_port ()) {
+                       continue;
+               }
+               if (s && i->second == s->midi_clock_output_port ()) {
+                       continue;
+               }
+               if (s && i->second == s->ltc_output_port ()) {
+                       continue;
+               }
                if (i->second->sends_output()) {
                        i->second->get_buffer(nframes).silence(nframes);
                }
@@ -685,9 +733,9 @@ void
 PortManager::check_monitoring ()
 {
        for (Ports::iterator i = _cycle_ports->begin(); i != _cycle_ports->end(); ++i) {
-               
+
                bool x;
-               
+
                if (i->second->last_monitor() != (x = i->second->monitoring_input ())) {
                        i->second->set_last_monitor (x);
                        /* XXX I think this is dangerous, due to
@@ -702,14 +750,14 @@ void
 PortManager::fade_out (gain_t base_gain, gain_t gain_step, pframes_t nframes)
 {
        for (Ports::iterator i = _cycle_ports->begin(); i != _cycle_ports->end(); ++i) {
-               
+
                if (i->second->sends_output()) {
-                       
+
                        boost::shared_ptr<AudioPort> ap = boost::dynamic_pointer_cast<AudioPort> (i->second);
                        if (ap) {
                                Sample* s = ap->engine_get_whole_audio_buffer ();
                                gain_t g = base_gain;
-                               
+
                                for (pframes_t n = 0; n < nframes; ++n) {
                                        *s++ *= g;
                                        g -= gain_step;