Fixed an MSVC compilation problem reported by Frederic Bouvier.

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

diff --git a/simgear/misc/props.cxx b/simgear/misc/props.cxx
index be18a0ddfc8c76fed740c95f7d0da4aa3e60723f..c198858bd5dcad9d58fad316bfd5680912eeec44 100644 (file)
--- a/simgear/misc/props.cxx
+++ b/simgear/misc/props.cxx
@@ -17,6 +17,7 @@
 #include <iostream>
 using std::cerr;
 using std::endl;
+using std::find;
 using std::sort;
 
 #else
@@ -25,6 +26,8 @@ using std::sort;
 #include <simgear/debug/logstream.hxx>
 
 SG_USING_STD(sort);
+SG_USING_STD(find);
+SG_USING_STD(vector);
 
 #endif
 
@@ -361,14 +364,14 @@ SGPropertyNode::set_bool (bool val)
 {
   if (_tied) {
     if (_value.bool_val->setValue(val)) {
-      firePropertyChange();
+      fireValueChanged();
       return true;
     } else {
       return false;
     }
   } else {
     _local_val.bool_val = val;
-    firePropertyChange();
+    fireValueChanged();
     return true;
   }
 }
@@ -378,14 +381,14 @@ SGPropertyNode::set_int (int val)
 {
   if (_tied) {
     if (_value.int_val->setValue(val)) {
-      firePropertyChange();
+      fireValueChanged();
       return true;
     } else {
       return false;
     }
   } else {
     _local_val.int_val = val;
-    firePropertyChange();
+    fireValueChanged();
     return true;
   }
 }
@@ -395,14 +398,14 @@ SGPropertyNode::set_long (long val)
 {
   if (_tied) {
     if (_value.long_val->setValue(val)) {
-      firePropertyChange();
+      fireValueChanged();
       return true;
     } else {
       return false;
     }
   } else {
     _local_val.long_val = val;
-    firePropertyChange();
+    fireValueChanged();
     return true;
   }
 }
@@ -412,14 +415,14 @@ SGPropertyNode::set_float (float val)
 {
   if (_tied) {
     if (_value.float_val->setValue(val)) {
-      firePropertyChange();
+      fireValueChanged();
       return true;
     } else {
       return false;
     }
   } else {
     _local_val.float_val = val;
-    firePropertyChange();
+    fireValueChanged();
     return true;
   }
 }
@@ -429,14 +432,14 @@ SGPropertyNode::set_double (double val)
 {
   if (_tied) {
     if (_value.double_val->setValue(val)) {
-      firePropertyChange();
+      fireValueChanged();
       return true;
     } else {
       return false;
     }
   } else {
     _local_val.double_val = val;
-    firePropertyChange();
+    fireValueChanged();
     return true;
   }
 }
@@ -446,7 +449,7 @@ SGPropertyNode::set_string (const char * val)
 {
   if (_tied) {
     if (_value.string_val->setValue(val)) {
-      firePropertyChange();
+      fireValueChanged();
       return true;
     } else {
       return false;
@@ -454,7 +457,7 @@ SGPropertyNode::set_string (const char * val)
   } else {
     delete [] _local_val.string_val;
     _local_val.string_val = copy_string(val);
-    firePropertyChange();
+    fireValueChanged();
     return true;
   }
 }
@@ -843,7 +846,7 @@ SGPropertyNode::getChild (const char * name, int index, bool create)
     SGPropertyNode_ptr node;
     pos = find_child(name, index, _removedChildren);
     if (pos >= 0) {
-      std::vector<SGPropertyNode_ptr>::iterator it = _removedChildren.begin();
+      vector<SGPropertyNode_ptr>::iterator it = _removedChildren.begin();
       it += pos;
       node = _removedChildren[pos];
       _removedChildren.erase(it);
@@ -852,9 +855,8 @@ SGPropertyNode::getChild (const char * name, int index, bool create)
       node = new SGPropertyNode(name, index, this);
     }
     _children.push_back(node);
-    if (_parent != 0)
-      _parent->firePropertyChange();
-    return _children[_children.size()-1];
+    fireChildAdded(node);
+    return node;
   } else {
     return 0;
   }
@@ -902,7 +904,7 @@ SGPropertyNode::removeChild (const char * name, int index, bool keep)
   SGPropertyNode_ptr ret;
   int pos = find_child(name, index, _children);
   if (pos >= 0) {
-    std::vector<SGPropertyNode_ptr>::iterator it = _children.begin();
+    vector<SGPropertyNode_ptr>::iterator it = _children.begin();
     it += pos;
     SGPropertyNode_ptr node = _children[pos];
     _children.erase(it);
@@ -911,9 +913,8 @@ SGPropertyNode::removeChild (const char * name, int index, bool keep)
     }
     node->setAttribute(REMOVED, true);
     ret = node;
+    fireChildRemoved(node);
   }
-  if (_parent != 0)
-    _parent->firePropertyChange();
   return ret;
 }
 
@@ -932,7 +933,8 @@ SGPropertyNode::getPath (bool simplify) const
     sprintf(buffer, "[%d]", _index);
     path += buffer;
   }
-  return path.c_str();
+  strncpy(_buffer, path.c_str(), MAX_STRING_LEN);
+  return _buffer;
 }
 
 SGPropertyNode::Type
@@ -1989,18 +1991,79 @@ SGPropertyNode::addChangeListener (SGPropertyChangeListener * listener)
   if (_listeners == 0)
     _listeners = new vector<SGPropertyChangeListener *>;
   _listeners->push_back(listener);
+  listener->register_property(this);
+}
+
+void
+SGPropertyNode::removeChangeListener (SGPropertyChangeListener * listener)
+{
+  vector<SGPropertyChangeListener *>::iterator it =
+    find(_listeners->begin(), _listeners->end(), listener);
+  if (it != _listeners->end()) {
+    _listeners->erase(it);
+    listener->unregister_property(this);
+    if (_listeners->empty()) {
+      vector<SGPropertyChangeListener *> * tmp = _listeners;
+      _listeners = 0;
+      delete tmp;
+    }
+  }
+}
+
+void
+SGPropertyNode::fireValueChanged ()
+{
+  fireValueChanged(this);
+}
+
+void
+SGPropertyNode::fireChildAdded (SGPropertyNode * child)
+{
+  fireChildAdded(this, child);
+}
+
+void
+SGPropertyNode::fireChildRemoved (SGPropertyNode * child)
+{
+  fireChildRemoved(this, child);
+}
+
+void
+SGPropertyNode::fireValueChanged (SGPropertyNode * node)
+{
+  if (_listeners != 0) {
+    for (int i = 0; i < _listeners->size(); i++) {
+      (*_listeners)[i]->valueChanged(node);
+    }
+  }
+  if (_parent != 0)
+    _parent->fireValueChanged(node);
+}
+
+void
+SGPropertyNode::fireChildAdded (SGPropertyNode * parent,
+                               SGPropertyNode * child)
+{
+  if (_listeners != 0) {
+    for (int i = 0; i < _listeners->size(); i++) {
+      (*_listeners)[i]->childAdded(parent, child);
+    }
+  }
+  if (_parent != 0)
+    _parent->fireChildAdded(parent, child);
 }
 
 void
-SGPropertyNode::firePropertyChange (SGPropertyNode * node)
+SGPropertyNode::fireChildRemoved (SGPropertyNode * parent,
+                                 SGPropertyNode * child)
 {
   if (_listeners != 0) {
     for (int i = 0; i < _listeners->size(); i++) {
-      (*_listeners)[i]->propertyChanged(node);
+      (*_listeners)[i]->childRemoved(parent, child);
     }
   }
   if (_parent != 0)
-    _parent->firePropertyChange(node);
+    _parent->fireChildRemoved(parent, child);
 }
 
 
@@ -2226,4 +2289,55 @@ SGPropertyNode_ptr::valid() const
 }
 
 
+\f
+////////////////////////////////////////////////////////////////////////
+// Implementation of SGPropertyChangeListener.
+////////////////////////////////////////////////////////////////////////
+
+SGPropertyChangeListener::~SGPropertyChangeListener ()
+{
+                               // This will come back and remove
+                               // the current item each time.  Is
+                               // that OK?
+  vector<SGPropertyNode *>::iterator it;
+  for (it = _properties.begin(); it != _properties.end(); it++)
+    (*it)->removeChangeListener(this);
+}
+
+void
+SGPropertyChangeListener::valueChanged (SGPropertyNode * node)
+{
+  // NO-OP
+}
+
+void
+SGPropertyChangeListener::childAdded (SGPropertyNode * node,
+                                     SGPropertyNode * child)
+{
+  // NO-OP
+}
+
+void
+SGPropertyChangeListener::childRemoved (SGPropertyNode * parent,
+                                       SGPropertyNode * child)
+{
+  // NO-OP
+}
+
+void
+SGPropertyChangeListener::register_property (SGPropertyNode * node)
+{
+  _properties.push_back(node);
+}
+
+void
+SGPropertyChangeListener::unregister_property (SGPropertyNode * node)
+{
+  vector<SGPropertyNode *>::iterator it =
+    find(_properties.begin(), _properties.end(), node);
+  if (it != _properties.end())
+    _properties.erase(it);
+}
+
+
 // end of props.cxx