Changeset 7086

Timestamp:

Jun 2, 2010, 6:47:57 PM (14 years ago)

Author:

landauf

Message:

Fixed a bug in PickupSpawner.

When one Pawn picked up the last item, spawnsRemaining_ became 0 and the PickupSpawner was destroyed immediately. But PickupSpawner::tick() continued iterating through the other pawns, which lead to a crash.

File:

• code/branches/presentation3/src/modules/pickup/PickupSpawner.cc (modified) (1 diff)

code/branches/presentation3/src/modules/pickup/PickupSpawner.cc

@@ -176 +176 @@
         if (this->isActive())
         {
+            SmartPtr<PickupSpawner> temp = this; // create a smart pointer to keep the PickupSpawner alive until we iterated through all Pawns (in case a Pawn takes the last pickup)
+            
             //! Iterate trough all Pawns.
             for (ObjectList<Pawn>::iterator it = ObjectList<Pawn>::begin(); it != ObjectList<Pawn>::end(); ++it)