Fixed a crash: the singleton needs to be instantiated the first time SGCommandMgr...

[simgear.git] / simgear / props / propertyObject.cxx

diff --git a/simgear/props/propertyObject.cxx b/simgear/props/propertyObject.cxx
index ff90ce77afdd3701630020ee39ad712a7643cf75..7a46056080ea67b153c27d53d4628d4c2abbd8cf 100644 (file)
--- a/simgear/props/propertyObject.cxx
+++ b/simgear/props/propertyObject.cxx
@@ -21,6 +21,8 @@
 
 #include "propertyObject.hxx"
 
+#include <simgear/structure/exception.hxx>
+
 namespace simgear
 {
 
@@ -30,62 +32,74 @@ void PropertyObjectBase::setDefaultRoot(SGPropertyNode* aRoot)
 {
   static_defaultRoot = aRoot;
 }
+  
+PropertyObjectBase::PropertyObjectBase() :
+  _path(NULL),
+  _prop(NULL)
+{
+    
+}
+
+PropertyObjectBase::PropertyObjectBase(const PropertyObjectBase& aOther) :
+  _path(aOther._path),
+  _prop(aOther._prop)
+{
+
+}
 
 PropertyObjectBase::PropertyObjectBase(const char* aChild) :
-  _base(NULL),
   _path(aChild),
   _prop(NULL)
 {
 }
   
 PropertyObjectBase::PropertyObjectBase(SGPropertyNode* aNode, const char* aChild) :
-  _base(aNode),
   _path(aChild),
-  _prop(NULL)
+  _prop(aNode)
 {
 
 }
   
 SGPropertyNode* PropertyObjectBase::node(bool aCreate) const
 {
-  if (_prop) {
+  if (_path == NULL) { // already resolved
     return _prop;
   }
   
-  _prop = _base ? _base : static_defaultRoot;
-  if (_path) {
-    _prop = _prop->getNode(_path, aCreate);
-  }
+  SGPropertyNode *r = _prop ? _prop : static_defaultRoot,
+                 *prop = r->getNode(_path, aCreate);
   
-  return _prop;
-}
+  if( prop )
+  {
+    // resolve worked, we will cache from now on, so clear _path and cache prop
+    _path = NULL;
+    _prop = prop;
+  }
 
-} // of namespace simgear
+  return prop;
+}
 
-void test()
+SGPropertyNode* PropertyObjectBase::getOrThrow() const
 {
-  SGPropObjDouble foo("/bar/foo");
+  SGPropertyNode* n = node(false);
+  if (!n) {
+    std::string path;
+    if (_prop) {
+      path = _prop->getPath();
+      if (_path) {
+        path += '/';      
+      }
+    }
 
-  SGPropertyNode* zoob;
+    if (_path) {
+      path += _path;
+    }
 
-  SGPropObjDouble foo2 = SGPropObjDouble::create(zoob, "foo2", 42.0);
-  
-  foo = 1123.0;
-  foo2 =  43;
-  
-  std::string s("lalala");
-  
-  foo = "lalal";
-  
-  
-  SGPropObjString sp(zoob);
-  sp = "fooo";
-  s =  sp;
-  
+    throw sg_exception("Unknown property:" + path);
+  }
 
-  SGPropObjBool bp("/some nice big path");
-  bp = false;
-  
-  bp = 456;
-  int i5 = bp;
+  return n;
 }
+
+} // of namespace simgear
+