Various tweaks to the bundle manager.
[ardour.git] / libs / ardour / io.cc
index a5d4e639e34e4bf5391c70d277d209c2bbf7b207..a56f81781483951c8ef64c39c97c3f7adf7f3585 100644 (file)
@@ -1,5 +1,5 @@
 /*
-    Copyright (C) 2000-2006 Paul Davis 
+    Copyright (C) 2000-2006 Paul Davis
 
     This program is free software; you can redistribute it and/or modify
     it under the terms of the GNU General Public License as published by
@@ -52,7 +52,7 @@
 #include <cmath>
 
 /*
-  A bug in OS X's cmath that causes isnan() and isinf() to be 
+  A bug in OS X's cmath that causes isnan() and isinf() to be
   "undeclared". the following works around that
 */
 
@@ -83,7 +83,6 @@ IO::IO (Session& s, const string& name, Direction dir, DataType default_type)
        _active = true;
        pending_state_node = 0;
        setup_bundles ();
-       cerr << "+++ IO created with name = " << _name << endl;
 }
 
 IO::IO (Session& s, const XMLNode& node, DataType dt)
@@ -94,10 +93,8 @@ IO::IO (Session& s, const XMLNode& node, DataType dt)
        _active = true;
        pending_state_node = 0;
 
-       set_state (node);
-
+       set_state (node, Stateful::loading_state_version);
        setup_bundles ();
-       cerr << "+++ IO created from XML with name = " << _name << endl;
 }
 
 IO::~IO ()
@@ -131,7 +128,7 @@ void
 IO::check_bundles (std::vector<UserBundleInfo>& list, const PortSet& ports)
 {
        std::vector<UserBundleInfo> new_list;
-       
+
        for (std::vector<UserBundleInfo>::iterator i = list.begin(); i != list.end(); ++i) {
 
                uint32_t const N = i->bundle->nchannels ();
@@ -175,20 +172,20 @@ IO::disconnect (Port* our_port, string other_port, void* src)
                return 0;
        }
 
-       { 
+       {
                BLOCK_PROCESS_CALLBACK ();
-               
+
                {
                        Glib::Mutex::Lock lm (io_lock);
-                       
+
                        /* check that our_port is really one of ours */
-                       
+
                        if ( ! _ports.contains(our_port)) {
                                return -1;
                        }
-                       
+
                        /* disconnect it from the source */
-                       
+
                        if (our_port->disconnect (other_port)) {
                                error << string_compose(_("IO: cannot disconnect port %1 from %2"), our_port->name(), other_port) << endmsg;
                                return -1;
@@ -213,16 +210,16 @@ IO::connect (Port* our_port, string other_port, void* src)
 
        {
                BLOCK_PROCESS_CALLBACK ();
-               
+
                {
                        Glib::Mutex::Lock lm (io_lock);
-                       
+
                        /* check that our_port is really one of ours */
-                       
+
                        if ( ! _ports.contains(our_port) ) {
                                return -1;
                        }
-                       
+
                        /* connect it to the source */
 
                        if (our_port->connect (other_port)) {
@@ -244,7 +241,7 @@ IO::remove_port (Port* port, void* src)
        {
                BLOCK_PROCESS_CALLBACK ();
 
-               
+
                {
                        Glib::Mutex::Lock lm (io_lock);
 
@@ -253,7 +250,7 @@ IO::remove_port (Port* port, void* src)
 
                                if (port->connected()) {
                                        change = IOChange (change|ConnectionsChanged);
-                               } 
+                               }
 
                                _session.engine().unregister_port (*port);
                                check_bundles_connected ();
@@ -263,7 +260,7 @@ IO::remove_port (Port* port, void* src)
                PortCountChanged (n_ports()); /* EMIT SIGNAL */
        }
 
-       if (change == ConfigurationChanged) {
+       if (change & ConfigurationChanged) {
                setup_bundles ();
        }
 
@@ -294,12 +291,12 @@ IO::add_port (string destination, void* src, DataType type)
        {
                BLOCK_PROCESS_CALLBACK ();
 
-               
-               { 
+
+               {
                        Glib::Mutex::Lock lm (io_lock);
-                       
+
                        /* Create a new output port */
-                       
+
                        string portname = build_legal_port_name (type);
 
                        if (_direction == Input) {
@@ -325,7 +322,7 @@ IO::add_port (string destination, void* src, DataType type)
                        return -1;
                }
        }
-       
+
        // pan_changed (src); /* EMIT SIGNAL */
        changed (ConfigurationChanged, src); /* EMIT SIGNAL */
        setup_bundles ();
@@ -337,12 +334,12 @@ IO::add_port (string destination, void* src, DataType type)
 int
 IO::disconnect (void* src)
 {
-       { 
+       {
                BLOCK_PROCESS_CALLBACK ();
-               
+
                {
                        Glib::Mutex::Lock lm (io_lock);
-                       
+
                        for (PortSet::iterator i = _ports.begin(); i != _ports.end(); ++i) {
                                i->disconnect_all ();
                        }
@@ -350,26 +347,26 @@ IO::disconnect (void* src)
                        check_bundles_connected ();
                }
        }
-       
+
        changed (ConnectionsChanged, src); /* EMIT SIGNAL */
-       
+
        return 0;
 }
 
 bool
-IO::ensure_ports_locked (ChanCount count, bool clear, void* src)
+IO::ensure_ports_locked (ChanCount count, bool clear, void* /*src*/)
 {
        Port* port = 0;
        bool  changed    = false;
-       
+
        for (DataType::iterator t = DataType::begin(); t != DataType::end(); ++t) {
-               
+
                const size_t n = count.get(*t);
-       
+
                /* remove unused ports */
                for (size_t i = n_ports().get(*t); i > n; --i) {
                        port = _ports.port(*t, i-1);
-                       
+
                        assert(port);
                        _ports.remove(port);
                        _session.engine().unregister_port (*port);
@@ -406,13 +403,13 @@ IO::ensure_ports_locked (ChanCount count, bool clear, void* src)
                        changed = true;
                }
        }
-       
+
        if (changed) {
                check_bundles_connected ();
                PortCountChanged (n_ports()); /* EMIT SIGNAL */
                _session.set_dirty ();
        }
-       
+
        if (clear) {
                /* disconnect all existing ports so that we get a fresh start */
                for (PortSet::iterator i = _ports.begin(); i != _ports.end(); ++i) {
@@ -429,11 +426,7 @@ IO::ensure_ports (ChanCount count, bool clear, bool lockit, void* src)
 {
        bool changed = false;
 
-       cerr << "Ensure that IO " << _name << '/' << (_direction == Input ? "input" : "output") 
-            << " has " << count << endl;
-
        if (count == n_ports() && !clear) {
-               cerr << "\talready has " << n_ports() << endl;
                return 0;
        }
 
@@ -451,8 +444,6 @@ IO::ensure_ports (ChanCount count, bool clear, bool lockit, void* src)
                _session.set_dirty ();
        }
 
-       cerr << "\t@" << this << "  established with " << n_ports() << endl;
-       
        return 0;
 }
 
@@ -469,7 +460,7 @@ IO::get_state (void)
 }
 
 XMLNode&
-IO::state (bool full_state)
+IO::state (bool /*full_state*/)
 {
        XMLNode* node = new XMLNode (state_node_name);
        char buf[64];
@@ -492,11 +483,12 @@ IO::state (bool full_state)
        }
 
        for (PortSet::iterator i = _ports.begin(); i != _ports.end(); ++i) {
-               
+
                vector<string> connections;
 
-               XMLNode* pnode = new XMLNode (X_("port"));
+               XMLNode* pnode = new XMLNode (X_("Port"));
                pnode->add_property (X_("type"), i->type().to_string());
+               pnode->add_property (X_("name"), i->name());
 
                if (i->get_connections (connections)) {
 
@@ -508,11 +500,14 @@ IO::state (bool full_state)
                                   to be re-established even when our
                                   client name is different.
                                */
-                               
-                               pnode->add_property (X_("connection"), _session.engine().make_port_name_relative (*ci));
-                       }       
+
+                               XMLNode* cnode = new XMLNode (X_("Connection"));
+
+                               cnode->add_property (X_("other"), _session.engine().make_port_name_relative (*ci));
+                               pnode->add_child_nocopy (*cnode);
+                       }
                }
-               
+
                node->add_child_nocopy (*pnode);
        }
 
@@ -520,8 +515,14 @@ IO::state (bool full_state)
 }
 
 int
-IO::set_state (const XMLNode& node)
+IO::set_state (const XMLNode& node, int version)
 {
+       /* callers for version < 3000 need to call set_state_2X directly, as A3 IOs
+        * are input OR output, not both, so the direction needs to be specified
+        * by the caller.
+        */
+       assert (version >= 3000);
+
        const XMLProperty* prop;
        XMLNodeConstIterator iter;
        LocaleGuard lg (X_("POSIX"));
@@ -534,7 +535,7 @@ IO::set_state (const XMLNode& node)
                error << string_compose(_("incorrect XML node \"%1\" passed to IO object"), node.name()) << endmsg;
                return -1;
        }
-       
+
        if ((prop = node.property ("name")) != 0) {
                set_name (prop->value());
        }
@@ -552,26 +553,77 @@ IO::set_state (const XMLNode& node)
                _direction = (Direction) string_2_enum (prop->value(), _direction);
        }
 
-       if (!connecting_legal) {
+       if (create_ports (node, version)) {
+               return -1;
+       }
+
+       if (connecting_legal) {
+
+               if (make_connections (node, version, false)) {
+                       return -1;
+               }
+
+       } else {
+
                pending_state_node = new XMLNode (node);
-       } 
+               pending_state_node_version = version;
+               pending_state_node_in = false;
+               connection_legal_c = ConnectingLegal.connect (mem_fun (*this, &IO::connecting_became_legal));
+       }
+
+
+       return 0;
+}
+
+int
+IO::set_state_2X (const XMLNode& node, int version, bool in)
+{
+       const XMLProperty* prop;
+       XMLNodeConstIterator iter;
+       LocaleGuard lg (X_("POSIX"));
+
+       /* force use of non-localized representation of decimal point,
+          since we use it a lot in XML files and so forth.
+       */
+
+       if (node.name() != state_node_name) {
+               error << string_compose(_("incorrect XML node \"%1\" passed to IO object"), node.name()) << endmsg;
+               return -1;
+       }
+
+       if ((prop = node.property ("name")) != 0) {
+               set_name (prop->value());
+       }
 
-       if (create_ports (node)) {
+       if ((prop = node.property (X_("default-type"))) != 0) {
+               _default_type = DataType(prop->value());
+               assert(_default_type != DataType::NIL);
+       }
+
+       if ((prop = node.property ("id")) != 0) {
+               _id = prop->value ();
+       }
+
+       _direction = in ? Input : Output;
+
+       if (create_ports (node, version)) {
                return -1;
        }
 
        if (connecting_legal) {
-               
-               if (make_connections (node)) {
+
+               if (make_connections_2X (node, version, in)) {
                        return -1;
                }
 
        } else {
-               
+
+               pending_state_node = new XMLNode (node);
+               pending_state_node_version = version;
+               pending_state_node_in = in;
                connection_legal_c = ConnectingLegal.connect (mem_fun (*this, &IO::connecting_became_legal));
        }
 
-
        return 0;
 }
 
@@ -584,8 +636,8 @@ IO::connecting_became_legal ()
 
        connection_legal_c.disconnect ();
 
-       ret = make_connections (*pending_state_node);
-       
+       ret = make_connections (*pending_state_node, pending_state_node_version, pending_state_node_in);
+
        delete pending_state_node;
        pending_state_node = 0;
 
@@ -598,7 +650,7 @@ IO::find_possible_bundle (const string &desired_name)
        static const string digits = "0123456789";
        const string &default_name = (_direction == Input ? _("in") : _("out"));
        const string &bundle_type_name = (_direction == Input ? _("input") : _("output"));
-       
+
        boost::shared_ptr<Bundle> c = _session.bundle_by_name (desired_name);
 
        if (!c) {
@@ -612,7 +664,7 @@ IO::find_possible_bundle (const string &desired_name)
 
                // find numeric suffix of desired name
                bundle_number = 0;
-               
+
                last_non_digit_pos = desired_name.find_last_not_of(digits);
 
                if (last_non_digit_pos != string::npos) {
@@ -620,7 +672,7 @@ IO::find_possible_bundle (const string &desired_name)
                        s << desired_name.substr(last_non_digit_pos);
                        s >> bundle_number;
                }
-       
+
                // see if it's a stereo connection e.g. "in 3+4"
 
                if (last_non_digit_pos > 1 && desired_name[last_non_digit_pos] == '+') {
@@ -648,22 +700,22 @@ IO::find_possible_bundle (const string &desired_name)
                // find highest set bit
                mask = 1;
                while ((mask <= bundle_number) && (mask <<= 1)) {}
-               
-               // "wrap" bundle number into largest possible power of 2 
+
+               // "wrap" bundle number into largest possible power of 2
                // that works...
 
                while (mask) {
 
                        if (bundle_number & mask) {
                                bundle_number &= ~mask;
-                               
+
                                stringstream s;
                                s << default_name << " " << bundle_number + 1;
 
                                if (stereo) {
                                        s << "+" << bundle_number + 2;
                                }
-                               
+
                                possible_name = s.str();
 
                                if ((c = _session.bundle_by_name (possible_name)) != 0) {
@@ -687,8 +739,36 @@ IO::find_possible_bundle (const string &desired_name)
 }
 
 int
-IO::get_port_counts (const XMLNode& node, ChanCount& n, boost::shared_ptr<Bundle>& c)
+IO::get_port_counts_2X (XMLNode const & node, int /*version*/, ChanCount& n, boost::shared_ptr<Bundle>& /*c*/)
 {
+       XMLProperty const * prop;
+       XMLNodeList children = node.children ();
+
+       uint32_t n_audio = 0;
+
+       for (XMLNodeIterator i = children.begin(); i != children.end(); ++i) {
+
+               if ((prop = node.property ("inputs")) != 0 && _direction == Input) {
+                       n_audio = count (prop->value().begin(), prop->value().end(), '{');
+               } else if ((prop = node.property ("outputs")) != 0 && _direction == Output) {
+                       n_audio = count (prop->value().begin(), prop->value().end(), '{');
+               }
+       }
+
+       ChanCount cnt;
+       cnt.set_audio (n_audio);
+       n = ChanCount::max (n, cnt);
+
+       return 0;
+}
+
+int
+IO::get_port_counts (const XMLNode& node, int version, ChanCount& n, boost::shared_ptr<Bundle>& c)
+{
+       if (version < 3000) {
+               return get_port_counts_2X (node, version, n, c);
+       }
+
        XMLProperty const * prop;
        XMLNodeConstIterator iter;
        uint32_t n_audio = 0;
@@ -704,7 +784,7 @@ IO::get_port_counts (const XMLNode& node, ChanCount& n, boost::shared_ptr<Bundle
                }
                return 0;
        }
-       
+
        for (iter = node.children().begin(); iter != node.children().end(); ++iter) {
 
                if ((*iter)->name() == X_("Bundle")) {
@@ -716,7 +796,7 @@ IO::get_port_counts (const XMLNode& node, ChanCount& n, boost::shared_ptr<Bundle
                        }
                }
 
-               if ((*iter)->name() == X_("port")) {
+               if ((*iter)->name() == X_("Port")) {
                        prop = (*iter)->property (X_("type"));
 
                        if (!prop) {
@@ -730,20 +810,18 @@ IO::get_port_counts (const XMLNode& node, ChanCount& n, boost::shared_ptr<Bundle
                        }
                }
        }
-       
+
        n = ChanCount::max (n, cnt);
        return 0;
 }
 
 int
-IO::create_ports (const XMLNode& node)
+IO::create_ports (const XMLNode& node, int version)
 {
        ChanCount n;
        boost::shared_ptr<Bundle> c;
-       
-       get_port_counts (node, n, c);
-       
-       cerr << _name << " got " << n << " from XML node" << endl;
+
+       get_port_counts (node, version, n, c);
 
        if (ensure_ports (n, true, true, this)) {
                error << string_compose(_("%1: cannot create I/O ports"), _name) << endmsg;
@@ -756,25 +834,13 @@ IO::create_ports (const XMLNode& node)
 }
 
 int
-IO::make_connections (const XMLNode& node)
+IO::make_connections (const XMLNode& node, int version, bool in)
 {
-       const XMLProperty* prop;
-
-       if ((prop = node.property ("connection")) != 0) {
-               boost::shared_ptr<Bundle> c = find_possible_bundle (prop->value());
-               
-               if (!c) {
-                       return -1;
-               }
-
-               if (n_ports().get(c->type()) == c->nchannels() && c->ports_are_outputs()) {
-                       connect_ports_to_bundle (c, this);
-               }
-
-               return 0;
-       } 
+       if (version < 3000) {
+               return make_connections_2X (node, version, in);
+       }
 
-       uint32_t n = 0;
+       const XMLProperty* prop;
 
        for (XMLNodeConstIterator i = node.children().begin(); i != node.children().end(); ++i) {
 
@@ -790,15 +856,132 @@ IO::make_connections (const XMLNode& node)
                        return 0;
                }
 
-               if ((*i)->name() == "port") {
-                       Port* p = nth (n++);
-                       XMLProperty* prop = (*i)->property ("connection");
-                       if (p && prop) {
-                               p->connect (prop->value());
+               if ((*i)->name() == "Port") {
+
+                       prop = (*i)->property (X_("name"));
+
+                       if (!prop) {
+                               continue;
+                       }
+
+                       Port* p = port_by_name (prop->value());
+
+                       if (p) {
+                               for (XMLNodeConstIterator c = (*i)->children().begin(); c != (*i)->children().end(); ++c) {
+
+                                       XMLNode* cnode = (*c);
+
+                                       if (cnode->name() != X_("Connection")) {
+                                               continue;
+                                       }
+
+                                       if ((prop = cnode->property (X_("other"))) == 0) {
+                                               continue;
+                                       }
+
+                                       if (prop) {
+                                               p->connect (prop->value());
+                                       }
+                               }
                        }
                }
        }
-       
+
+       return 0;
+}
+
+
+int
+IO::make_connections_2X (const XMLNode& node, int /*version*/, bool in)
+{
+       const XMLProperty* prop;
+
+       /* XXX: bundles ("connections" as was) */
+
+       if ((prop = node.property ("inputs")) != 0 && in) {
+
+               string::size_type ostart = 0;
+               string::size_type start = 0;
+               string::size_type end = 0;
+               int i = 0;
+               int n;
+               vector<string> ports;
+
+               string const str = prop->value ();
+
+               while ((start = str.find_first_of ('{', ostart)) != string::npos) {
+                       start += 1;
+
+                       if ((end = str.find_first_of ('}', start)) == string::npos) {
+                               error << string_compose(_("IO: badly formed string in XML node for inputs \"%1\""), str) << endmsg;
+                               return -1;
+                       }
+
+                       if ((n = parse_io_string (str.substr (start, end - start), ports)) < 0) {
+                               error << string_compose(_("bad input string in XML node \"%1\""), str) << endmsg;
+
+                               return -1;
+
+                       } else if (n > 0) {
+
+
+                               for (int x = 0; x < n; ++x) {
+                                       /* XXX: this is a bit of a hack; need to check if it's always valid */
+                                       string::size_type const p = ports[x].find ("/out");
+                                       if (p != string::npos) {
+                                               ports[x].replace (p, 4, "/audio_out");
+                                       }
+                                       nth(i)->connect (ports[x]);
+                               }
+                       }
+
+                       ostart = end+1;
+                       i++;
+               }
+
+       }
+
+       if ((prop = node.property ("outputs")) != 0 && !in) {
+
+               string::size_type ostart = 0;
+               string::size_type start = 0;
+               string::size_type end = 0;
+               int i = 0;
+               int n;
+               vector<string> ports;
+
+               string const str = prop->value ();
+
+               while ((start = str.find_first_of ('{', ostart)) != string::npos) {
+                       start += 1;
+
+                       if ((end = str.find_first_of ('}', start)) == string::npos) {
+                               error << string_compose(_("IO: badly formed string in XML node for outputs \"%1\""), str) << endmsg;
+                               return -1;
+                       }
+
+                       if ((n = parse_io_string (str.substr (start, end - start), ports)) < 0) {
+                               error << string_compose(_("IO: bad output string in XML node \"%1\""), str) << endmsg;
+
+                               return -1;
+
+                       } else if (n > 0) {
+
+                               for (int x = 0; x < n; ++x) {
+                                       /* XXX: this is a bit of a hack; need to check if it's always valid */
+                                       string::size_type const p = ports[x].find ("/in");
+                                       if (p != string::npos) {
+                                               ports[x].replace (p, 3, "/audio_in");
+                                       }
+                                       nth(i)->connect (ports[x]);
+                               }
+                       }
+
+                       ostart = end+1;
+                       i++;
+               }
+       }
+
        return 0;
 }
 
@@ -809,7 +992,7 @@ IO::set_ports (const string& str)
        int i;
        int n;
        uint32_t nports;
-       
+
        if ((nports = count (str.begin(), str.end(), '{')) == 0) {
                return 0;
        }
@@ -838,7 +1021,7 @@ IO::set_ports (const string& str)
                        error << string_compose(_("bad input string in XML node \"%1\""), str) << endmsg;
 
                        return -1;
-                       
+
                } else if (n > 0) {
 
                        for (int x = 0; x < n; ++x) {
@@ -871,7 +1054,7 @@ IO::parse_io_string (const string& str, vector<string>& ports)
                ports.push_back (str.substr (opos, pos - opos));
                opos = pos + 1;
        }
-       
+
        if (opos < str.length()) {
                ports.push_back (str.substr(opos));
        }
@@ -892,7 +1075,7 @@ IO::parse_gain_string (const string& str, vector<string>& ports)
                ports.push_back (str.substr (opos, pos - opos));
                opos = pos + 1;
        }
-       
+
        if (opos < str.length()) {
                ports.push_back (str.substr(opos));
        }
@@ -903,18 +1086,11 @@ IO::parse_gain_string (const string& str, vector<string>& ports)
 bool
 IO::set_name (const string& requested_name)
 {
-       if (requested_name == _name) {
+       string name = requested_name;
+
+       if (name == _name) {
                return true;
        }
-       
-       string name;
-       Route *rt;
-       if ( (rt = dynamic_cast<Route *>(this))) {
-               name = Route::ensure_track_or_route_name(requested_name, _session);
-       } else {
-               name = requested_name;
-       }
-
 
        /* replace all colons in the name. i wish we didn't have to do this */
 
@@ -928,7 +1104,7 @@ IO::set_name (const string& requested_name)
                i->set_name (current_name);
        }
 
-       bool const r = SessionObject::set_name(name);
+       bool const r = SessionObject::set_name (name);
 
        setup_bundles ();
 
@@ -958,7 +1134,7 @@ IO::latency () const
        for (PortSet::const_iterator i = _ports.begin(); i != _ports.end(); ++i) {
                if ((latency = i->total_latency ()) > max_latency) {
                        max_latency = latency;
-               } 
+               }
        }
 
        return max_latency;
@@ -1013,7 +1189,7 @@ IO::disconnect_ports_from_bundle (boost::shared_ptr<Bundle> c, void* src)
                Glib::Mutex::Lock lm2 (io_lock);
 
                c->disconnect (_bundle, _session.engine());
-                       
+
                /* If this is a UserBundle, make a note of what we've done */
 
                boost::shared_ptr<UserBundle> ub = boost::dynamic_pointer_cast<UserBundle> (c);
@@ -1050,7 +1226,7 @@ IO::enable_connecting ()
 }
 
 void
-IO::bundle_changed (Bundle::Change c)
+IO::bundle_changed (Bundle::Change /*c*/)
 {
        //XXX
 //     connect_input_ports_to_bundle (_input_bundle, this);
@@ -1071,11 +1247,17 @@ IO::build_legal_port_name (DataType type)
        } else {
                throw unknown_type();
        }
-       
+
+       /* note that if "in" or "out" are translated it will break a session
+          across locale switches because a port's connection list will
+          show (old) translated names, but the current port name will
+          use the (new) translated name.
+       */
+
        if (_direction == Input) {
-               suffix += _("_in");
+               suffix += X_("_in");
        } else {
-               suffix += _("_out");
+               suffix += X_("_out");
        }
 
        // allow up to 4 digits for the output port number, plus the slash, suffix and extra space
@@ -1084,9 +1266,9 @@ IO::build_legal_port_name (DataType type)
 
        char buf1[name_size+1];
        char buf2[name_size+1];
-       
+
        snprintf (buf1, name_size+1, ("%.*s/%s"), limit, _name.c_str(), suffix.c_str());
-       
+
        int port_number = find_port_hole (buf1);
        snprintf (buf2, name_size+1, "%s %d", buf1, port_number);
 
@@ -1150,7 +1332,7 @@ IO::setup_bundles ()
         char buf[32];
 
        if (!_bundle) {
-               _bundle.reset (new Bundle (true));
+               _bundle.reset (new Bundle (_direction == Input));
        }
 
        _bundle->suspend_signals ();
@@ -1179,7 +1361,7 @@ BundleList
 IO::bundles_connected ()
 {
        BundleList bundles;
-       
+
        /* User bundles */
        for (std::vector<UserBundleInfo>::iterator i = _bundles_connected.begin(); i != _bundles_connected.end(); ++i) {
                bundles.push_back (i->bundle);
@@ -1210,7 +1392,7 @@ IO::bundles_connected ()
                        }
                }
        }
-         
+
        return bundles;
 }
 
@@ -1227,7 +1409,7 @@ std::string
 IO::bundle_channel_name (uint32_t c, uint32_t n) const
 {
        char buf[32];
-       
+
        switch (n) {
        case 1:
                return _("mono");
@@ -1245,11 +1427,11 @@ string
 IO::name_from_state (const XMLNode& node)
 {
        const XMLProperty* prop;
-       
+
        if ((prop = node.property ("name")) != 0) {
                return prop->value();
-       } 
-       
+       }
+
        return string();
 }
 
@@ -1257,21 +1439,33 @@ void
 IO::set_name_in_state (XMLNode& node, const string& new_name)
 {
        const XMLProperty* prop;
-       
+
        if ((prop = node.property ("name")) != 0) {
                node.add_property ("name", new_name);
-       } 
+       }
 }
 
 bool
 IO::connected_to (boost::shared_ptr<const IO> other) const
 {
+       if (!other) {
+               /* do we have any connections at all? */
+
+               for (PortSet::const_iterator p = _ports.begin(); p != _ports.end(); ++p) {
+                       if (p->connected()) {
+                               return true;
+                       }
+               }
+
+               return false;
+       }
+
        assert (_direction != other->direction());
 
        uint32_t i, j;
        uint32_t no = n_ports().n_total();
        uint32_t ni = other->n_ports ().n_total();
-       
+
        for (i = 0; i < no; ++i) {
                for (j = 0; j < ni; ++j) {
                        if (nth(i)->connected_to (other->nth(j)->name())) {
@@ -1291,14 +1485,14 @@ IO::process_input (boost::shared_ptr<Processor> proc, sframes_t start_frame, sfr
        /* don't read the data into new buffers - just use the port buffers directly */
 
        bufs.attach_buffers (_ports, nframes, 0);
-       proc->run_in_place (bufs, start_frame, end_frame, nframes);
+       proc->run (bufs, start_frame, end_frame, nframes);
 }
 
 void
 IO::collect_input (BufferSet& bufs, nframes_t nframes, ChanCount offset)
 {
        assert(bufs.available() >= _ports.count());
-       
+
        if (_ports.count() == ChanCount::ZERO) {
                return;
        }
@@ -1326,7 +1520,7 @@ void
 IO::copy_to_outputs (BufferSet& bufs, DataType type, nframes_t nframes, nframes_t offset)
 {
        // Copy any buffers 1:1 to outputs
-       
+
        PortSet::iterator o = _ports.begin(type);
        BufferSet::iterator i = bufs.begin(type);
        BufferSet::iterator prev = i;
@@ -1338,7 +1532,7 @@ IO::copy_to_outputs (BufferSet& bufs, DataType type, nframes_t nframes, nframes_
                ++i;
                ++o;
        }
-       
+
        // Copy last buffer to any extra outputs
 
        while (o != _ports.end(type)) {
@@ -1347,3 +1541,20 @@ IO::copy_to_outputs (BufferSet& bufs, DataType type, nframes_t nframes, nframes_
                ++o;
        }
 }
+
+Port*
+IO::port_by_name (const std::string& str) const
+{
+       /* to be called only from ::set_state() - no locking */
+
+       for (PortSet::const_iterator i = _ports.begin(); i != _ports.end(); ++i) {
+
+               const Port& p(*i);
+
+               if (p.name() == str) {
+                       return const_cast<Port*>(&p);
+               }
+       }
+
+       return 0;
+}