Some platforms don't include libxml++config.h automatically.

[libcxml.git] / src / cxml.cc

diff --git a/src/cxml.cc b/src/cxml.cc
index 922a3d713a45849d2078438ecfb39b1ee61915fc..d4eb1388499a284f6333faec165def73e7b977b6 100644 (file)
--- a/src/cxml.cc
+++ b/src/cxml.cc
@@ -20,17 +20,21 @@
 
 #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 (0)
+       : _node (nullptr)
 {
 
 }
@@ -83,9 +87,9 @@ cxml::Node::node_children () const
                throw Error ("No node to read children from");
        }
 
-       vector<shared_ptr<cxml::Node> > n;
+       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;
@@ -98,14 +102,20 @@ cxml::Node::node_children (string name) const
           how get_path works.
        */
 
-       vector<shared_ptr<cxml::Node> > n;
+       if (!_node) {
+               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;
 }
 
@@ -208,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());
                }
        }
@@ -241,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)
 {
@@ -290,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 ();
@@ -456,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 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);
+}