more or less complete restoration of Controllable::_id from XML, with all that implie...

[ardour.git] / libs / pbd / controllable.cc

diff --git a/libs/pbd/controllable.cc b/libs/pbd/controllable.cc
index fbbb4d6f254a51d44d59952e5cddb5fd0ec89c3a..0112ff2b1ccabf76200fdcf30a94e24a196eeac9 100644 (file)
--- a/libs/pbd/controllable.cc
+++ b/libs/pbd/controllable.cc
@@ -18,95 +18,65 @@
 */
 
 #include "pbd/controllable.h"
+#include "pbd/enumwriter.h"
 #include "pbd/xml++.h"
 #include "pbd/error.h"
+#include "pbd/locale_guard.h"
 
 #include "i18n.h"
 
 using namespace PBD;
+using namespace std;
 
-sigc::signal<void,Controllable*> Controllable::Destroyed;
-sigc::signal<bool,Controllable*> Controllable::StartLearning;
-sigc::signal<void,Controllable*> Controllable::StopLearning;
-sigc::signal<void,Controllable*,int,int> Controllable::CreateBinding;
-sigc::signal<void,Controllable*> Controllable::DeleteBinding;
+PBD::Signal1<void,Controllable*> Controllable::Destroyed;
+PBD::Signal1<bool,Controllable*> Controllable::StartLearning;
+PBD::Signal1<void,Controllable*> Controllable::StopLearning;
+PBD::Signal3<void,Controllable*,int,int> Controllable::CreateBinding;
+PBD::Signal1<void,Controllable*> Controllable::DeleteBinding;
 
 Glib::StaticRWLock Controllable::registry_lock = GLIBMM_STATIC_RW_LOCK_INIT;
 Controllable::Controllables Controllable::registry;
-Controllable::ControllablesByURI Controllable::registry_by_uri;
+PBD::ScopedConnectionList registry_connections;
+const std::string Controllable::xml_node_name = X_("Controllable");
 
-Controllable::Controllable (const std::string& name, const std::string& uri)
+Controllable::Controllable (const string& name, Flag f)
        : _name (name)
-       , _uri (uri)
+       , _flags (f)
        , _touching (false)
 {
-       add ();
+       add (*this);
 }
 
 void
-Controllable::add ()
+Controllable::add (Controllable& ctl)
 {
-       Glib::RWLock::WriterLock lm (*registry_lock);
-       registry.insert (this);
-
-       if (!_uri.empty()) {
-               pair<string,Controllable*> newpair;
-               newpair.first = _uri;
-               newpair.second = this;
-               registry_by_uri.insert (newpair);
-       }
+       using namespace boost;
+
+       Glib::RWLock::WriterLock lm (registry_lock);
+       registry.insert (&ctl);
+
+       /* Controllable::remove() is static - no need to manage this connection */
 
-       this->GoingAway.connect (mem_fun (this, &Controllable::remove));
+       ctl.DropReferences.connect_same_thread (registry_connections, boost::bind (&Controllable::remove, &ctl));
 }
 
 void
-Controllable::remove ()
+Controllable::remove (Controllable* ctl)
 {
-       Glib::RWLock::WriterLock lm (*registry_lock);
+       Glib::RWLock::WriterLock lm (registry_lock);
 
        for (Controllables::iterator i = registry.begin(); i != registry.end(); ++i) {
-               if ((*i) == this) {
+               if ((*i) == ctl) {
                        registry.erase (i);
                        break;
                }
        }
-
-       if (_uri) {
-               ControllablesByURI::iterator i = registry_by_uri.find (_uri);
-               if (i != registry_by_uri.end()) {
-                       registry_by_uri.erase (i);
-               }
-       }
-
-
-}
-
-void
-Controllable::set_uri (const std::string& new_uri)
-{
-       Glib::RWLock::WriterLock lm (*registry_lock);
-
-       if (_uri) {
-               ControllablesByURI::iterator i = registry_by_uri.find (_uri);
-               if (i != registry_by_uri.end()) {
-                       registry_by_uri.erase (i);
-               }
-       }
-
-       _uri = new_uri;
-
-       if (!_uri.empty()) {
-               pair<string,Controllable*> newpair;
-               newpair.first = _uri;
-               newpair.second = this;
-               registry_by_uri.insert (newpair);
-       }
 }
 
 Controllable*
 Controllable::by_id (const ID& id)
 {
-       Glib::RWLock::ReaderLock lm (*registry_lock);
+       Glib::RWLock::ReaderLock lm (registry_lock);
 
        for (Controllables::iterator i = registry.begin(); i != registry.end(); ++i) {
                if ((*i)->id() == id) {
@@ -117,21 +87,9 @@ Controllable::by_id (const ID& id)
 }
 
 Controllable*
-Controllable::by_uri (const string& uri)
+Controllable::by_name (const string& str)
 {
-       Glib::RWLock::ReaderLock lm (*registry_lock);
-       ControllablesByURI::iterator i;
-
-       if ((i = registry_by_ui.find (uri)) != registry_by_uri.end()) {
-               return i->second;
-       }
-       return 0;
-}
-
-Controllable*
-Controllable::by_name (const std::string& str)
-{
-       Glib::RWLock::ReaderLock lm (*registry_lock);
+       Glib::RWLock::ReaderLock lm (registry_lock);
 
        for (Controllables::iterator i = registry.begin(); i != registry.end(); ++i) {
                if ((*i)->_name == str) {
@@ -144,34 +102,51 @@ Controllable::by_name (const std::string& str)
 XMLNode&
 Controllable::get_state ()
 {
-       XMLNode* node = new XMLNode (X_("Controllable"));
+       XMLNode* node = new XMLNode (xml_node_name);
+       LocaleGuard lg (X_("POSIX"));
        char buf[64];
 
        node->add_property (X_("name"), _name); // not reloaded from XML state, just there to look at
        _id.print (buf, sizeof (buf));
        node->add_property (X_("id"), buf);
+       node->add_property (X_("flags"), enum_2_string (_flags));
+       snprintf (buf, sizeof (buf), "%2.12f", get_value());
+        node->add_property (X_("value"), buf);
 
-       if (!_uri.empty()) {
-               node->add_property (X_("uri"), _uri);
-       }
-               
        return *node;
 }
 
+
 int
 Controllable::set_state (const XMLNode& node, int /*version*/)
 {
+       LocaleGuard lg (X_("POSIX"));
        const XMLProperty* prop;
 
        if ((prop = node.property (X_("id"))) != 0) {
                _id = prop->value();
-               return 0;
        } else {
                error << _("Controllable state node has no ID property") << endmsg;
                return -1;
        }
 
-       if ((prop = node.property (X_("uri"))) != 0) {
-               set_uri (prop->value());
+       if ((prop = node.property (X_("flags"))) != 0) {
+               _flags = (Flag) string_2_enum (prop->value(), _flags);
        }
+
+       if ((prop = node.property (X_("value"))) != 0) {
+               float val;
+
+               if (sscanf (prop->value().c_str(), "%f", &val) == 1) {
+                       set_value (val);
+               } 
+        }
+
+        return 0;
+}
+
+void
+Controllable::set_flags (Flag f)
+{
+       _flags = f;
 }