Some platforms don't even have libxml++ version defines.

[libcxml.git] / src / cxml.cc

diff --git a/src/cxml.cc b/src/cxml.cc
index c2d85347d2a90cd893dc483439520b4826168989..c6ab5cce1a41f009796fa17eeb0da50b773d88a9 100644 (file)
--- a/src/cxml.cc
+++ b/src/cxml.cc
@@ -20,15 +20,19 @@
 
 #include "cxml.h"
 #include <libxml++/libxml++.h>
+#include <libxml++config.h>
 #include <boost/filesystem.hpp>
 #include <boost/algorithm/string.hpp>
 #include <cstdio>
 
+
+using std::make_shared;
 using std::shared_ptr;
 using std::string;
 using std::vector;
 using boost::optional;
 
+
 cxml::Node::Node ()
        : _node (nullptr)
 {
@@ -85,7 +89,7 @@ cxml::Node::node_children () const
 
        vector<shared_ptr<cxml::Node>> n;
        for (auto i: _node->get_children()) {
-               n.push_back (shared_ptr<Node> (new Node (i)));
+               n.push_back(make_shared<Node>(i));
        }
 
        return n;
@@ -102,14 +106,16 @@ cxml::Node::node_children (string name) const
                throw cxml::Error("Node has no internal xmlpp node; did you forget to call a read method on cxml::Document?");
        }
 
+       auto const glib_name = Glib::ustring(name);
+
        vector<shared_ptr<cxml::Node>> n;
        for (auto i: _node->get_children()) {
-               if (i->get_name() == name) {
-                       n.push_back (shared_ptr<Node> (new Node (i)));
+               if (i->get_name() == glib_name) {
+                       n.push_back(make_shared<Node>(i));
                }
        }
 
-       _taken.push_back (name);
+       _taken.push_back(glib_name);
        return n;
 }
 
@@ -212,7 +218,7 @@ void
 cxml::Node::done () const
 {
        for (auto i: _node->get_children()) {
-               if (dynamic_cast<xmlpp::Element *> (i) && find (_taken.begin(), _taken.end(), i->get_name()) == _taken.end ()) {
+               if (dynamic_cast<xmlpp::Element*>(i) && find(_taken.begin(), _taken.end(), i->get_name()) == _taken.end()) {
                        throw cxml::Error ("unexpected XML node " + i->get_name());
                }
        }
@@ -245,6 +251,13 @@ cxml::Node::namespace_prefix () const
        return _node->get_namespace_prefix ();
 }
 
+
+bool
+cxml::Node::is_text() const
+{
+       return dynamic_cast<const xmlpp::TextNode*>(_node);
+}
+
 cxml::Document::Document (string root_name)
        : _root_name (root_name)
 {
@@ -294,7 +307,7 @@ cxml::Document::take_root_node ()
        }
 
        _node = _parser->get_document()->get_root_node ();
-       if (!_root_name.empty() && _node->get_name() != _root_name) {
+       if (!_root_name.empty() && _node->get_name() != Glib::ustring(_root_name)) {
                throw cxml::Error ("unrecognised root node " + _node->get_name() + " (expecting " + _root_name + ")");
        } else if (_root_name.empty ()) {
                _root_name = _node->get_name ();
@@ -460,3 +473,21 @@ cxml::raw_convert (string v)
 {
        return locale_convert<double> (make_local(v));
 }
+
+
+xmlpp::Element*
+cxml::add_child(xmlpp::Element* parent, string const& name, string const& ns_prefix)
+{
+#if !defined(LIBXMLXX_MAJOR_VERSION) || LIBXMLXX_MAJOR_VERSION == 2
+       return parent->add_child(name, ns_prefix);
+#else
+       return parent->add_child_element(name, ns_prefix);
+#endif
+}
+
+
+void
+cxml::add_text_child(xmlpp::Element* parent, string const& name, string const& text)
+{
+       add_child(parent, name)->add_child_text(text);
+}