gras: we now have the element tree interface

4 files changed, 61 insertions, 64 deletions

diff --git a/lib/block_props.cpp b/lib/block_props.cpp
index ab8b642..672200b 100644
--- a/lib/block_props.cpp
+++ b/lib/block_props.cpp
@@ -2,8 +2,6 @@
 
 #include "element_impl.hpp"
 #include <gras/block.hpp>
-#include <boost/foreach.hpp>
-#include <boost/algorithm/string.hpp>
 
 using namespace gras;
 
@@ -76,51 +74,8 @@ struct PropAccessReceiver : Theron::Receiver
  * Handle the get and set calls from the user's call-stack
  **********************************************************************/
 template <typename ActorType>
-static PMCC prop_access_dispatcher(ActorType &actor, const std::string &path, const PMCC &value, const bool set)
+static PMCC prop_access_dispatcher(ActorType &actor, const std::string &key, const PMCC &value, const bool set)
 {
-    //split the paths into nodes
-    std::vector<std::string> nodes;
-    boost::split(nodes, path, boost::is_any_of("/"));
-
-    //iterate through the path to find the element
-    std::string key;
-    Element elem = *(actor->block_ptr);
-    size_t i = 0;
-    BOOST_FOREACH(const std::string &node, nodes)
-    {
-        i++;
-        if (node == "" and i == 1) //find root
-        {
-            while (elem->_parent) elem = elem->_parent;
-            continue;
-        }
-        if (node == ".") //this dir
-        {
-            continue;
-        }
-        if (node == "..") //up a dir
-        {
-            if (not elem->_parent) throw std::runtime_error(
-                "Property tree lookup fail - null parent: " + path
-            );
-            elem = elem->_parent;
-            continue;
-        }
-        if (i == nodes.size() or elem->block) //leaf
-        {
-            key = node;
-            if (i != nodes.size() or not elem->block) throw std::runtime_error(
-                "Property tree lookup fail - beyond leaf: " + path
-            );
-            continue;
-        }
-        if (elem->_subelems.count(node) == 0) throw std::runtime_error(
-            "Property tree lookup fail - no such path: " + path
-        );
-        elem = elem->_subelems[node];
-    }
-
-    //now send a message to the actor to perform the action
     PropAccessReceiver receiver;
     PropAccessMessage message;
     message.set = set;
diff --git a/lib/element.cpp b/lib/element.cpp
index 368dbea..e6c8499 100644
--- a/lib/element.cpp
+++ b/lib/element.cpp
@@ -4,6 +4,8 @@
 #include <gras/element.hpp>
 #include <boost/format.hpp>
 #include <boost/detail/atomic_count.hpp>
+#include <boost/foreach.hpp>
+#include <boost/algorithm/string.hpp>
 
 static boost::detail::atomic_count unique_id_pool(0);
 
@@ -45,3 +47,58 @@ std::string Element::to_string(void) const
 {
     return (*this)->id;
 }
+
+void Element::adopt_element(const std::string &name, const Element &child)
+{
+    if (child->parent) throw std::invalid_argument(str(boost::format(
+        "Could not register child %s into %s.\n"
+        "The child %s already has parent %s.\n"
+    )
+        % child.to_string()
+        % this->to_string()
+        % child.to_string()
+        % child->parent.to_string()
+    ));
+    //TODO check name not already taken
+    child->parent = *this;
+    (*this)->children[name] = child;
+}
+
+Block *Element::lookup_block(const std::string &path)
+{
+    //split the paths into nodes
+    std::vector<std::string> nodes;
+    boost::split(nodes, path, boost::is_any_of("/"));
+
+    //iterate through the path to find the element
+    boost::shared_ptr<ElementImpl> elem = *this;
+    size_t i = 0;
+    BOOST_FOREACH(const std::string &node, nodes)
+    {
+        i++;
+        if (node == "" and i == 1) //find root
+        {
+            while (elem->parent) elem = elem->parent;
+            continue;
+        }
+        if (node == ".") //this dir
+        {
+            continue;
+        }
+        if (node == "..") //up a dir
+        {
+            if (not elem->parent) throw std::invalid_argument(
+                "Element tree lookup fail - null parent: " + path
+            );
+            elem = elem->parent;
+            continue;
+        }
+        if (elem->children.count(node) == 0) throw std::invalid_argument(
+            "Element tree lookup fail - no such path: " + path
+        );
+        elem = elem->children[node];
+    }
+
+    //return block ptr as result
+    return elem->block->block_ptr;
+}
diff --git a/lib/element_impl.hpp b/lib/element_impl.hpp
index ccdbb54..869006e 100644
--- a/lib/element_impl.hpp
+++ b/lib/element_impl.hpp
@@ -35,9 +35,9 @@ struct ElementImpl
     Token token;
     GlobalBlockConfig top_config;
 
-    //hier block stuff
-    Element _parent;
-    std::map<std::string, Element> _subelems;
+    //element tree stuff
+    Element parent;
+    std::map<std::string, Element> children;
 
     //things may be in this element
     boost::shared_ptr<Apology::Topology> topology;
diff --git a/lib/hier_block.cpp b/lib/hier_block.cpp
index cfaca86..b7ffe4c 100644
--- a/lib/hier_block.cpp
+++ b/lib/hier_block.cpp
@@ -76,18 +76,3 @@ void HierBlock::commit(void)
 {
     (*this)->topology->commit();
 }
-
-void HierBlock::register_subelement(const std::string &node, const Element &subelem)
-{
-    if (subelem->_parent) throw std::runtime_error(str(boost::format(
-        "Could not register subelement %s into %s.\n"
-        "The subelement %s already has parent %s.\n"
-    )
-        % subelem.to_string()
-        % this->to_string()
-        % subelem.to_string()
-        % subelem->_parent.to_string()
-    ));
-    subelem->_parent = *this;
-    (*this)->_subelems[node] = subelem;
-}