Changeset 10417 for code/branches/core7/src/modules

Timestamp:

May 3, 2015, 1:31:18 PM (10 years ago)

Author:

landauf

Message:

moved destruction of bullet collision shape to ~CollisionShape(). CompoundCollisionShape needs some trickery to avoid that a shape gets deleted twice because it does some magic (i.e. assignments) in CompoundCollisionShape::updatePublicShape()

Location:

code/branches/core7/src/modules/objects/collisionshapes

Files:

• BoxCollisionShape.cc (modified) (1 diff)
• BoxCollisionShape.h (modified) (1 diff)
• ConeCollisionShape.cc (modified) (1 diff)
• ConeCollisionShape.h (modified) (1 diff)
• CylinderCollisionShape.cc (modified) (1 diff)
• CylinderCollisionShape.h (modified) (1 diff)
• PlaneCollisionShape.cc (modified) (1 diff)
• PlaneCollisionShape.h (modified) (1 diff)
• SphereCollisionShape.cc (modified) (1 diff)
• SphereCollisionShape.h (modified) (1 diff)

code/branches/core7/src/modules/objects/collisionshapes/BoxCollisionShape.cc

@@ -58 +58 @@
     }
 
-    BoxCollisionShape::~BoxCollisionShape()
-    {
-        // TODO: Move to CollisionShape?
-        if (this->isInitialized())
-            delete this->collisionShape_;
-    }
-
     void BoxCollisionShape::registerVariables()
     {

code/branches/core7/src/modules/objects/collisionshapes/BoxCollisionShape.h

@@ -58 +58 @@
         public:
             BoxCollisionShape(Context* context);
-            virtual ~BoxCollisionShape();
 
             virtual void XMLPort(Element& xmlelement, XMLPort::Mode mode);

code/branches/core7/src/modules/objects/collisionshapes/ConeCollisionShape.cc

@@ -54 +54 @@
     }
 
-    ConeCollisionShape::~ConeCollisionShape()
-    {
-        if (this->isInitialized())
-            delete this->collisionShape_;
-    }
-
     /**
     @brief

code/branches/core7/src/modules/objects/collisionshapes/ConeCollisionShape.h

@@ -56 +56 @@
         public:
             ConeCollisionShape(Context* context);
-            virtual ~ConeCollisionShape();
 
         private:

code/branches/core7/src/modules/objects/collisionshapes/CylinderCollisionShape.cc

@@ -54 +54 @@
     }
 
-    CylinderCollisionShape::~CylinderCollisionShape()
-    {
-        if (this->isInitialized())
-            delete this->collisionShape_;
-    }
-
     /**
     @brief

code/branches/core7/src/modules/objects/collisionshapes/CylinderCollisionShape.h

@@ -53 +53 @@
         public:
             CylinderCollisionShape(Context* context);
-            virtual ~CylinderCollisionShape();
 
         private:

code/branches/core7/src/modules/objects/collisionshapes/PlaneCollisionShape.cc

@@ -59 +59 @@
     }
 
-    PlaneCollisionShape::~PlaneCollisionShape()
-    {
-        if (this->isInitialized())
-            delete this->collisionShape_;
-    }
-
     void PlaneCollisionShape::registerVariables()
     {

code/branches/core7/src/modules/objects/collisionshapes/PlaneCollisionShape.h

@@ -58 +58 @@
         public:
             PlaneCollisionShape(Context* context);
-            virtual ~PlaneCollisionShape();
 
             virtual void XMLPort(Element& xmlelement, XMLPort::Mode mode);

code/branches/core7/src/modules/objects/collisionshapes/SphereCollisionShape.cc

@@ -58 +58 @@
     }
 
-    SphereCollisionShape::~SphereCollisionShape()
-    {
-        if (this->isInitialized())
-            delete this->collisionShape_;
-    }
-
     void SphereCollisionShape::registerVariables()
     {

code/branches/core7/src/modules/objects/collisionshapes/SphereCollisionShape.h

@@ -56 +56 @@
         public:
             SphereCollisionShape(Context* context);
-            virtual ~SphereCollisionShape();
 
             virtual void XMLPort(Element& xmlelement, XMLPort::Mode mode);