Commit 38564d77 authored by louiz’'s avatar louiz’

Little cleanup of the XmlNode class

Use map instead of unordered map (it's not slower, and it's shorter).
Use the default move constructor.
parent d7e1214c
#ifndef XMPP_STANZA_INCLUDED
# define XMPP_STANZA_INCLUDED
#include <unordered_map>
#include <map>
#include <string>
#include <vector>
......@@ -24,18 +24,9 @@ class XmlNode
public:
explicit XmlNode(const std::string& name, XmlNode* parent);
explicit XmlNode(const std::string& name);
XmlNode(XmlNode&& node):
name(std::move(node.name)),
parent(node.parent),
attributes(std::move(node.attributes)),
children(std::move(node.children)),
inner(std::move(node.inner)),
tail(std::move(node.tail))
{
node.parent = nullptr;
}
XmlNode(XmlNode&& node) = default;
/**
* The copy constructor do not copy the parent attribute. The children
* The copy constructor does not copy the parent attribute. The children
* nodes are all copied recursively.
*/
XmlNode(const XmlNode& node):
......@@ -134,7 +125,7 @@ public:
private:
std::string name;
XmlNode* parent;
std::unordered_map<std::string, std::string> attributes;
std::map<std::string, std::string> attributes;
std::vector<XmlNode*> children;
std::string inner;
std::string tail;
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment