Use XMLNode::get/set_property API in ARDOUR::VCA class
authorTim Mayberry <mojofunk@gmail.com>
Sun, 28 Aug 2016 12:07:50 +0000 (22:07 +1000)
committerTim Mayberry <mojofunk@gmail.com>
Tue, 18 Apr 2017 23:36:53 +0000 (09:36 +1000)
libs/ardour/vca.cc

index e3476c5d082b16ed0916d72e5afe6518a6656d2d..d056c855edd607a604f001f4d2557928cece4601 100644 (file)
@@ -114,8 +114,8 @@ XMLNode&
 VCA::get_state ()
 {
        XMLNode* node = new XMLNode (xml_node_name);
-       node->add_property (X_("name"), _name);
-       node->add_property (X_("number"), _number);
+       node->set_property (X_("name"), name());
+       node->set_property (X_("number"), _number);
 
        node->add_child_nocopy (_presentation_info.get_state());
 
@@ -132,35 +132,30 @@ VCA::get_state ()
 int
 VCA::set_state (XMLNode const& node, int version)
 {
-       XMLProperty const* prop;
-
        Stripable::set_state (node, version);
 
-       if ((prop = node.property ("name")) != 0) {
-               set_name (prop->value());
+       std::string str;
+       if (node.get_property ("name", str)) {
+               set_name (str);
        }
 
-       if ((prop = node.property ("number")) != 0) {
-               _number = atoi (prop->value());
-       }
+       node.get_property ("number", _number);
 
        XMLNodeList const &children (node.children());
        for (XMLNodeList::const_iterator i = children.begin(); i != children.end(); ++i) {
                if ((*i)->name() == Controllable::xml_node_name) {
 
-                       XMLProperty* prop = (*i)->property ("name");
-
-                       if (!prop) {
+                       if (!(*i)->get_property ("name", str)) {
                                continue;
                        }
 
-                       if (prop->value() == _gain_control->name()) {
+                       if (str == _gain_control->name()) {
                                _gain_control->set_state (**i, version);
                        }
-                       if (prop->value() == _solo_control->name()) {
+                       if (str == _solo_control->name()) {
                                _solo_control->set_state (**i, version);
                        }
-                       if (prop->value() == _mute_control->name()) {
+                       if (str == _mute_control->name()) {
                                _mute_control->set_state (**i, version);
                        }
                } else if ((*i)->name() == Slavable::xml_node_name) {