Modified Files:

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

diff --git a/simgear/props/props.cxx b/simgear/props/props.cxx
index 90efffed704cd2950b082f2ab38788d5ff8fa385..3ce2687da62d906a14f56e7a877024c78765432e 100644 (file)
--- a/simgear/props/props.cxx
+++ b/simgear/props/props.cxx
@@ -758,8 +758,9 @@ SGPropertyNode::~SGPropertyNode ()
   clearValue();
 
   if (_listeners) {
-    for (unsigned i = 0; i < _listeners.size(); ++i)
-      _listeners[i]->unregister_property(this);
+    vector<SGPropertyChangeListener*>::iterator it;
+    for (it = _listeners->begin(); it != _listeners->end(); ++it)
+      (*it)->unregister_property(this);
     delete _listeners;
   }
 }
@@ -2285,12 +2286,8 @@ SGPropertyNode::hash_table::hashcode (const char * key)
 
 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);
+  for (int i = _properties.size() - 1; i >= 0; i--)
+    _properties[i]->removeChangeListener(this);
 }
 
 void