Switched memory to using shared nodes, and node_data to keep only naked node...

Switched memory to using shared nodes, and node_data to keep only naked node pointers, not shared nodes (to break the cycle, and we don't need weak pointers because their memory is guaranteed to exist, via 'memory')

include/yaml-cpp/value/detail/impl.h

@@ -15,21 +15,21 @@ namespace YAML
 	{
 		// indexing
 		template<typename Key>
-		inline shared_node node_data::get(const Key& key, shared_memory_holder pMemory) const
+		inline node& node_data::get(const Key& key, shared_memory_holder pMemory) const
 		{
 			if(m_type != ValueType::Map)
 				return pMemory->create_node();
 			
 			for(node_map::const_iterator it=m_map.begin();it!=m_map.end();++it) {
-				if(equals(it->first, key, pMemory))
-					return it->second;
+				if(equals(*it->first, key, pMemory))
+					return *it->second;
 			}
 			
 			return pMemory->create_node();
 		}
 		
 		template<typename Key>
-		inline shared_node node_data::get(const Key& key, shared_memory_holder pMemory)
+		inline node& node_data::get(const Key& key, shared_memory_holder pMemory)
 		{
 			// TODO: check if 'key' is index-like, and we're a sequence
 			
@@ -48,14 +48,14 @@ namespace YAML
 			}
 			
 			for(node_map::const_iterator it=m_map.begin();it!=m_map.end();++it) {
-				if(equals(it->first, key, pMemory))
-					return it->second;
+				if(equals(*it->first, key, pMemory))
+					return *it->second;
 			}
 			
-			shared_node pKey = convert_to_node(key, pMemory);
-			shared_node pValue = pMemory->create_node();
-			m_map.push_back(kv_pair(pKey, pValue));
-			return pValue;
+			node& k = convert_to_node(key, pMemory);
+			node& v = pMemory->create_node();
+			m_map.push_back(kv_pair(&k, &v));
+			return v;
 		}
 		
 		template<typename Key>
@@ -65,7 +65,7 @@ namespace YAML
 				return false;
 			
 			for(node_map::iterator it=m_map.begin();it!=m_map.end();++it) {
-				if(equals(it->first, key, pMemory)) {
+				if(equals(*it->first, key, pMemory)) {
 					m_map.erase(it);
 					return true;
 				}
@@ -75,20 +75,20 @@ namespace YAML
 		}
 
 		template<typename T>
-		inline bool node_data::equals(detail::shared_node pNode, const T& rhs, detail::shared_memory_holder pMemory)
+		inline bool node_data::equals(node& node, const T& rhs, shared_memory_holder pMemory)
 		{
 			T lhs;
-			if(convert<T>::decode(Value(pNode, pMemory), lhs))
+			if(convert<T>::decode(Value(node, pMemory), lhs))
 				return lhs == rhs;
 			return false;
 		}
 		
 		template<typename T>
-		inline shared_node node_data::convert_to_node(const T& rhs, detail::shared_memory_holder pMemory)
+		inline node& node_data::convert_to_node(const T& rhs, shared_memory_holder pMemory)
 		{
 			Value value = convert<T>::encode(rhs);
 			pMemory->merge(*value.m_pMemory);
-			return value.m_pNode;
+			return *value.m_pNode;
 		}
 	}
 }

include/yaml-cpp/value/detail/memory.h

@@ -15,11 +15,11 @@ namespace YAML
 	{
 		class memory {
 		public:
-			shared_node create_node();
+			node& create_node();
 			void merge(const memory& rhs);
 			
 		private:
-			typedef std::set<shared_node_ref> Nodes;
+			typedef std::set<shared_node> Nodes;
 			Nodes m_nodes;
 		};
 
@@ -27,7 +27,7 @@ namespace YAML
 		public:
 			memory_holder(): m_pMemory(new memory) {}
 			
-			shared_node create_node() { return m_pMemory->create_node(); }
+			node& create_node() { return m_pMemory->create_node(); }
 			void merge(memory_holder& rhs);
 			
 		private:

include/yaml-cpp/value/detail/node.h

@@ -10,15 +10,16 @@
 #include "yaml-cpp/value/type.h"
 #include "yaml-cpp/value/ptr.h"
 #include "yaml-cpp/value/detail/node_ref.h"
+#include <boost/utility.hpp>
 
 namespace YAML
 {
 	namespace detail
 	{
-		class node
+		class node: private boost::noncopyable
 		{
 		public:
-			explicit node(shared_node_ref pRef): m_pRef(pRef) {}
+			node(): m_pRef(new node_ref) {}
 
 			bool is(const node& rhs) const { return m_pRef == rhs.m_pRef; }
 			ValueType::value type() const { return m_pRef->type(); }
@@ -33,13 +34,13 @@ namespace YAML
 			void set_scalar(const std::string& scalar) { m_pRef->set_scalar(scalar); }
 
 			// indexing
-			template<typename Key> shared_node get(const Key& key, shared_memory_holder pMemory) const { return static_cast<const node_ref&>(*m_pRef).get(key, pMemory); }
-			template<typename Key> shared_node get(const Key& key, shared_memory_holder pMemory) { return m_pRef->get(key, pMemory); }
+			template<typename Key> node& get(const Key& key, shared_memory_holder pMemory) const { return static_cast<const node_ref&>(*m_pRef).get(key, pMemory); }
+			template<typename Key> node& get(const Key& key, shared_memory_holder pMemory) { return m_pRef->get(key, pMemory); }
 			template<typename Key> bool remove(const Key& key, shared_memory_holder pMemory) { return m_pRef->remove(key, pMemory); }
 			
-			shared_node get(shared_node pKey, shared_memory_holder pMemory) const { return static_cast<const node_ref&>(*m_pRef).get(pKey, pMemory); }
-			shared_node get(shared_node pKey, shared_memory_holder pMemory) { return m_pRef->get(pKey, pMemory); }
-			bool remove(shared_node pKey, shared_memory_holder pMemory) { return m_pRef->remove(pKey, pMemory); }
+			node& get(node& key, shared_memory_holder pMemory) const { return static_cast<const node_ref&>(*m_pRef).get(key, pMemory); }
+			node& get(node& key, shared_memory_holder pMemory) { return m_pRef->get(key, pMemory); }
+			bool remove(node& key, shared_memory_holder pMemory) { return m_pRef->remove(key, pMemory); }
 
 		private:
 			shared_node_ref m_pRef;

include/yaml-cpp/value/detail/node_data.h

@@ -9,6 +9,7 @@
 #include "yaml-cpp/dll.h"
 #include "yaml-cpp/value/ptr.h"
 #include "yaml-cpp/value/type.h"
+#include <boost/utility.hpp>
 #include <list>
 #include <utility>
 #include <vector>
@@ -17,7 +18,7 @@ namespace YAML
 {
 	namespace detail
 	{
-		class node_data
+		class node_data: private boost::noncopyable
 		{
 		public:
 			node_data();
@@ -30,22 +31,22 @@ namespace YAML
 			const std::string& scalar() const { return m_scalar; }
 			
 			// indexing
-			template<typename Key> shared_node get(const Key& key, shared_memory_holder pMemory) const;
-			template<typename Key> shared_node get(const Key& key, shared_memory_holder pMemory);
+			template<typename Key> node& get(const Key& key, shared_memory_holder pMemory) const;
+			template<typename Key> node& get(const Key& key, shared_memory_holder pMemory);
 			template<typename Key> bool remove(const Key& key, shared_memory_holder pMemory);
 			
-			shared_node get(shared_node pKey, shared_memory_holder pMemory) const;
-			shared_node get(shared_node pKey, shared_memory_holder pMemory);
-			bool remove(shared_node pKey, shared_memory_holder pMemory);
+			node& get(node& key, shared_memory_holder pMemory) const;
+			node& get(node& key, shared_memory_holder pMemory);
+			bool remove(node& key, shared_memory_holder pMemory);
 			
 		private:
 			void convert_sequence_to_map(shared_memory_holder pMemory);
 			
 			template<typename T>
-			static bool equals(detail::shared_node pNode, const T& rhs, detail::shared_memory_holder pMemory);
+			static bool equals(node& node, const T& rhs, shared_memory_holder pMemory);
 			
 			template<typename T>
-			static shared_node convert_to_node(const T& rhs, detail::shared_memory_holder pMemory);
+			static node& convert_to_node(const T& rhs, shared_memory_holder pMemory);
 
 		private:
 			bool m_isDefined;
@@ -55,11 +56,11 @@ namespace YAML
 			std::string m_scalar;
 			
 			// sequence
-			typedef std::vector<shared_node> node_seq;
+			typedef std::vector<node *> node_seq;
 			node_seq m_sequence;
 			
 			// map
-			typedef std::pair<shared_node, shared_node> kv_pair;
+			typedef std::pair<node *, node *> kv_pair;
 			typedef std::list<kv_pair> node_map;
 			node_map m_map;
 		};

include/yaml-cpp/value/detail/node_ref.h

@@ -10,12 +10,13 @@
 #include "yaml-cpp/value/type.h"
 #include "yaml-cpp/value/ptr.h"
 #include "yaml-cpp/value/detail/node_data.h"
+#include <boost/utility.hpp>
 
 namespace YAML
 {
 	namespace detail
 	{
-		class node_ref
+		class node_ref: private boost::noncopyable
 		{
 		public:
 			node_ref(): m_pData(new node_data) {}
@@ -30,13 +31,13 @@ namespace YAML
 			void set_scalar(const std::string& scalar) { m_pData->set_scalar(scalar); }
 			
 			// indexing
-			template<typename Key> shared_node get(const Key& key, shared_memory_holder pMemory) const { return static_cast<const node_data&>(*m_pData).get(key, pMemory); }
-			template<typename Key> shared_node get(const Key& key, shared_memory_holder pMemory) { return m_pData->get(key, pMemory); }
+			template<typename Key> node& get(const Key& key, shared_memory_holder pMemory) const { return static_cast<const node_data&>(*m_pData).get(key, pMemory); }
+			template<typename Key> node& get(const Key& key, shared_memory_holder pMemory) { return m_pData->get(key, pMemory); }
 			template<typename Key> bool remove(const Key& key, shared_memory_holder pMemory) { return m_pData->remove(key, pMemory); }
 			
-			shared_node get(shared_node pKey, shared_memory_holder pMemory) const { return static_cast<const node_data&>(*m_pData).get(pKey, pMemory); }
-			shared_node get(shared_node pKey, shared_memory_holder pMemory) { return m_pData->get(pKey, pMemory); }
-			bool remove(shared_node pKey, shared_memory_holder pMemory) { return m_pData->remove(pKey, pMemory); }
+			node& get(node& key, shared_memory_holder pMemory) const { return static_cast<const node_data&>(*m_pData).get(key, pMemory); }
+			node& get(node& key, shared_memory_holder pMemory) { return m_pData->get(key, pMemory); }
+			bool remove(node& key, shared_memory_holder pMemory) { return m_pData->remove(key, pMemory); }
 			
 		private:
 			shared_node_data m_pData;

include/yaml-cpp/value/impl.h

@@ -13,18 +13,18 @@
 
 namespace YAML
 {
-	inline Value::Value(): m_pMemory(new detail::memory_holder), m_pNode(m_pMemory->create_node())
+	inline Value::Value(): m_pMemory(new detail::memory_holder), m_pNode(&m_pMemory->create_node())
 	{
 		m_pNode->set_null();
 	}
 	
-	inline Value::Value(ValueType::value type): m_pMemory(new detail::memory_holder), m_pNode(m_pMemory->create_node())
+	inline Value::Value(ValueType::value type): m_pMemory(new detail::memory_holder), m_pNode(&m_pMemory->create_node())
 	{
 		m_pNode->set_type(type);
 	}
 	
 	template<typename T>
-	inline Value::Value(const T& rhs): m_pMemory(new detail::memory_holder), m_pNode(m_pMemory->create_node())
+	inline Value::Value(const T& rhs): m_pMemory(new detail::memory_holder), m_pNode(&m_pMemory->create_node())
 	{
 		Assign(rhs);
 	}
@@ -33,7 +33,7 @@ namespace YAML
 	{
 	}
 	
-	inline Value::Value(detail::shared_node pNode, detail::shared_memory_holder pMemory): m_pMemory(pMemory), m_pNode(pNode)
+	inline Value::Value(detail::node& node, detail::shared_memory_holder pMemory): m_pMemory(pMemory), m_pNode(&node)
 	{
 	}
 
@@ -154,15 +154,15 @@ namespace YAML
 	template<typename Key>
 	inline const Value Value::operator[](const Key& key) const
 	{
-		detail::shared_node pValue = static_cast<const detail::node&>(*m_pNode).get(key, m_pMemory);
-		return Value(pValue, m_pMemory);
+		detail::node& value = static_cast<const detail::node&>(*m_pNode).get(key, m_pMemory);
+		return Value(value, m_pMemory);
 	}
 	
 	template<typename Key>
 	inline Value Value::operator[](const Key& key)
 	{
-		detail::shared_node pValue = m_pNode->get(key, m_pMemory);
-		return Value(pValue, m_pMemory);
+		detail::node& value = m_pNode->get(key, m_pMemory);
+		return Value(value, m_pMemory);
 	}
 	
 	template<typename Key>
@@ -173,19 +173,19 @@ namespace YAML
 	
 	inline const Value Value::operator[](const Value& key) const
 	{
-		detail::shared_node pValue = static_cast<const detail::node&>(*m_pNode).get(key.m_pNode, m_pMemory);
-		return Value(pValue, m_pMemory);
+		detail::node& value = static_cast<const detail::node&>(*m_pNode).get(*key.m_pNode, m_pMemory);
+		return Value(value, m_pMemory);
 	}
 	
 	inline Value Value::operator[](const Value& key)
 	{
-		detail::shared_node pValue = m_pNode->get(key.m_pNode, m_pMemory);
-		return Value(pValue, m_pMemory);
+		detail::node& value = m_pNode->get(*key.m_pNode, m_pMemory);
+		return Value(value, m_pMemory);
 	}
 	
 	inline bool Value::remove(const Value& key)
 	{
-		return m_pNode->remove(key.m_pNode, m_pMemory);
+		return m_pNode->remove(*key.m_pNode, m_pMemory);
 	}
 	
 	inline const Value Value::operator[](const char *key) const

include/yaml-cpp/value/value.h

@@ -63,7 +63,7 @@ namespace YAML
 		bool remove(char *key);
 		
 	private:
-		explicit Value(detail::shared_node pNode, detail::shared_memory_holder pMemory);
+		explicit Value(detail::node& node, detail::shared_memory_holder pMemory);
 		
 		template<typename T> void Assign(const T& rhs);
 		void Assign(const char *rhs);
@@ -74,7 +74,7 @@ namespace YAML
 		
 	private:
 		detail::shared_memory_holder m_pMemory;
-		detail::shared_node m_pNode;
+		detail::node *m_pNode;
 	};
 	
 	int compare(const Value& lhs, const Value& rhs);

src/value/detail/memory.cpp

@@ -14,11 +14,11 @@ namespace YAML
 			rhs.m_pMemory = m_pMemory;
 		}
 		
-		shared_node memory::create_node()
+		node& memory::create_node()
 		{
-			shared_node_ref pRef(new node_ref);
-			m_nodes.insert(pRef);
-			return shared_node(new node(pRef));
+			shared_node pNode(new node);
+			m_nodes.insert(pNode);
+			return *pNode;
 		}
 		
 		void memory::merge(const memory& rhs)

src/value/detail/node_data.cpp

@@ -58,20 +58,20 @@ namespace YAML
 		}
 
 		// indexing
-		shared_node node_data::get(shared_node pKey, shared_memory_holder pMemory) const
+		node& node_data::get(node& key, shared_memory_holder pMemory) const
 		{
 			if(m_type != ValueType::Map)
 				return pMemory->create_node();
 			
 			for(node_map::const_iterator it=m_map.begin();it!=m_map.end();++it) {
-				if(it->first == pKey)
-					return it->second;
+				if(it->first == &key) // TODO: equality?
+					return *it->second;
 			}
 			
 			return pMemory->create_node();
 		}
 		
-		shared_node node_data::get(shared_node pKey, shared_memory_holder pMemory)
+		node& node_data::get(node& key, shared_memory_holder pMemory)
 		{
 			switch(m_type) {
 				case ValueType::Undefined:
@@ -88,22 +88,22 @@ namespace YAML
 			}
 
 			for(node_map::const_iterator it=m_map.begin();it!=m_map.end();++it) {
-				if(it->first == pKey)
-					return it->second;
+				if(it->first == &key) // TODO: equality?
+					return *it->second;
 			}
 			
-			shared_node pValue = pMemory->create_node();
-			m_map.push_back(kv_pair(pKey, pValue));
-			return pValue;
+			node& value = pMemory->create_node();
+			m_map.push_back(kv_pair(&key, &value));
+			return value;
 		}
 		
-		bool node_data::remove(shared_node pKey, shared_memory_holder /* pMemory */)
+		bool node_data::remove(node& key, shared_memory_holder /* pMemory */)
 		{
 			if(m_type != ValueType::Map)
 				return false;
 
 			for(node_map::iterator it=m_map.begin();it!=m_map.end();++it) {
-				if(it->first == pKey) {
+				if(it->first == &key) { // TODO: equality?
 					m_map.erase(it);
 					return true;
 				}
@@ -121,9 +121,9 @@ namespace YAML
 				std::stringstream stream;
 				stream << i;
 
-				shared_node pKey = pMemory->create_node();
-				pKey->set_scalar(stream.str());
-				m_map.push_back(kv_pair(pKey, m_sequence[i]));
+				node& key = pMemory->create_node();
+				key.set_scalar(stream.str());
+				m_map.push_back(kv_pair(&key, m_sequence[i]));
 			}
 			
 			m_sequence.clear();