Changeset 9294 for code/branches/presentation2012merge/src/modules/pickup/PickupCollection.cc

Timestamp:

Jun 12, 2012, 11:05:15 PM (12 years ago)

Author:

landauf

Message:

removed 2 lines from ~PickupCollection() again because they don't solve the real issue
use destroy() for PickupCollectionIdentifier because it's an OrxonoxClass
calling this→destroy() should be the last thing you do in a function
details in PickupIdentifier

File:

• code/branches/presentation2012merge/src/modules/pickup/PickupCollection.cc (modified) (3 diffs)

code/branches/presentation2012merge/src/modules/pickup/PickupCollection.cc

@@ -75 +75 @@
         for(std::list<CollectiblePickup*>::iterator it = this->pickups_.begin(); it != this->pickups_.end(); ++it)
         {
-            if((*it)->isBeingDestroyed())
-                continue;
-
             (*it)->wasRemovedFromCollection();
             (*it)->destroyPickup();
@@ -84 +81 @@
 
         if(this->pickupCollectionIdentifier_ != NULL)
-            delete this->pickupCollectionIdentifier_;
+            this->pickupCollectionIdentifier_->destroy();
     }
 
@@ -187 +184 @@
         if(this->pickedUpCounter_ <= this->disabledCounter_ && this->isPickedUp())
             this->Pickupable::destroy();
-
-        // If the PickupCollection is no longer picked up.
-        if(!this->isPickedUp())
+        else if(!this->isPickedUp()) // If the PickupCollection is no longer picked up.
             this->pickedUpCounter_ = 0;
     }