Changeset 6497 for code/branches/pickup3/src/modules/pickup

Timestamp:

Mar 8, 2010, 9:17:22 PM (15 years ago)

Author:

dafrick

Message:

Fixed bug in clone() (Argument needed to be as reference). Pickups seem to be working now (after very sporadic testing), more systematic testing (and probably still some more bug fixes) will follow.

Location:

code/branches/pickup3/src/modules/pickup

Files:

• Pickup.cc (modified) (2 diffs)
• Pickup.h (modified) (1 diff)
• PickupCollection.cc (modified) (1 diff)
• PickupCollection.h (modified) (1 diff)
• items/HealthPickup.cc (modified) (1 diff)
• items/HealthPickup.h (modified) (1 diff)

code/branches/pickup3/src/modules/pickup/Pickup.cc

@@ -188 +188 @@
         SUPER(Pickup, changedCarrier);
         
+        COUT(1) << "Changed Carrier: " << this->isPickedUp() << this->isImmediate() << std::endl;
         //! Sets the Pickup to used if the Pickup has activation type 'immediate' and gets picked up.
         if(this->getCarrier() != NULL && this->isPickedUp() && this->isImmediate())
@@ -201 +202 @@
         Returns the clone of this pickup as a pointer to a Pickupable.
     */
-    void Pickup::clone(OrxonoxClass* item)
+    void Pickup::clone(OrxonoxClass*& item)
     {
         if(item == NULL)

code/branches/pickup3/src/modules/pickup/Pickup.h

@@ -119 +119 @@
             virtual void changedCarrier(void); //!< Should be called when the pickup has transited from picked up to dropped or the other way around.
                                     
-            virtual void clone(OrxonoxClass* item); //!< Creates a duplicate of the Pickup.
+            virtual void clone(OrxonoxClass*& item); //!< Creates a duplicate of the Pickup.
                 
         protected:

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

@@ -161 +161 @@
     
     //TODO: Steal description from Pickupable.
-    void PickupCollection::clone(OrxonoxClass* item)
+    void PickupCollection::clone(OrxonoxClass*& item)
     {
         if(item == NULL)

code/branches/pickup3/src/modules/pickup/PickupCollection.h

@@ -62 +62 @@
             virtual void changedCarrier(void);
             
-            virtual void clone(OrxonoxClass* item);
+            virtual void clone(OrxonoxClass*& item);
             
             virtual const PickupIdentifier* getPickupIdentifier(void);

code/branches/pickup3/src/modules/pickup/items/HealthPickup.cc

@@ -280 +280 @@
         A pointer to the Orxonox class.
     */
-    void HealthPickup::clone(OrxonoxClass* item)
+    void HealthPickup::clone(OrxonoxClass*& item)
     {
         if(item == NULL)

code/branches/pickup3/src/modules/pickup/items/HealthPickup.h

@@ -74 +74 @@
             
             virtual void changedUsed(void); //!< Is called when the pickup has transited from used to unused or the other way around.
-            virtual void clone(OrxonoxClass* item); //!< Creates a duplicate of the input OrxonoxClass.
+            virtual void clone(OrxonoxClass*& item); //!< Creates a duplicate of the input OrxonoxClass.
             
             /**