Use XMLNode::get/set_property in ARDOUR::ChanMapping class

This avoids possible demotion of unsigned integers when using the
add_property(char*, long) API. Which is unlikely to have ever been an issue but
worth noting.

diff --git a/libs/ardour/chan_mapping.cc b/libs/ardour/chan_mapping.cc
index f6f0e092c8065acd309bfe7d92c1cf73fb7be600..b3cbb10ef40c680a7cd956eaa00ee447f7b5020d 100644 (file)
--- a/libs/ardour/chan_mapping.cc
+++ b/libs/ardour/chan_mapping.cc
@@ -22,6 +22,7 @@
 #include <stdint.h>
 #include <iostream>
 #include "ardour/chan_mapping.h"
+#include "ardour/types_convert.h"
 
 #include "pbd/i18n.h"
 
@@ -59,10 +60,13 @@ ChanMapping::ChanMapping (const XMLNode& node)
        XMLNodeConstIterator iter = node.children().begin();
        for ( ; iter != node.children().end(); ++iter) {
                if ((*iter)->name() == X_(state_node_name)) {
-                       const string& type_str  = (*iter)->property("type")->value();
-                       const string& from_str = (*iter)->property("from")->value();
-                       const string& to_str = (*iter)->property("to")->value();
-                       set(DataType(type_str), atol (from_str.c_str()), atol (to_str.c_str()));
+                       DataType type(DataType::NIL);
+                       uint32_t from;
+                       uint32_t to;
+                       (*iter)->get_property("type", type);
+                       (*iter)->get_property("from", from);
+                       (*iter)->get_property("to", to);
+                       set(type, from, to);
                }
        }
 }
@@ -160,9 +164,9 @@ ChanMapping::state(const std::string& name) const
        for (Mappings::const_iterator tm = mp.begin(); tm != mp.end(); ++tm) {
                for (TypeMapping::const_iterator i = tm->second.begin(); i != tm->second.end(); ++i) {
                        XMLNode* n = new XMLNode(X_(state_node_name));
-                       n->add_property("type", tm->first.to_string());
-                       n->add_property("from", i->first);
-                       n->add_property("to", i->second);
+                       n->set_property("type", tm->first.to_string());
+                       n->set_property("from", i->first);
+                       n->set_property("to", i->second);
                        node->add_child_nocopy(*n);
                }
        }