Changeset 5211 in orxonox.OLD for trunk/src/lib

Timestamp:

Sep 21, 2005, 2:21:41 PM (19 years ago)

Author:

bensch

Message:

orxonox/trunk: valgrind mem-leak-recovered

Location:

trunk/src/lib

Files:

• coord/p_node.cc (modified) (1 diff)
• graphics/importer/model.cc (modified) (1 diff)
• graphics/importer/texture.cc (modified) (1 diff)
• graphics/light.cc (modified) (1 diff)
• graphics/render2D/element_2d.cc (modified) (2 diffs)
• graphics/render2D/element_2d.h (modified) (1 diff)
• math/curve.cc (modified) (9 diffs)
• physics/physics_engine.cc (modified) (1 diff)
• util/list.h (modified) (1 diff)

trunk/src/lib/coord/p_node.cc

@@ -104 +104 @@
 {
   this->setClassID(CL_PARENT_NODE, "PNode");
+
   this->children = new tList<PNode>();
   this->bRelCoorChanged = true;

trunk/src/lib/graphics/importer/model.cc

@@ -350 +350 @@
     delete this->normals;
   if (this->triangles)
-    delete this->triangles;
+    delete[] this->triangles;
 
   this->vertices = NULL;
   this->vTexture = NULL;
   this->normals = NULL;
-  this->triangles = NULL; this->triangleCount = 0;
+  this->triangles = NULL;
+  this->triangleCount = 0;
 }
 

trunk/src/lib/graphics/importer/texture.cc

@@ -45 +45 @@
 Texture::~Texture()
 {
-  if (this->texture)
+  if (this->texture != 0)
     glDeleteTextures(1, &this->texture);
 }

trunk/src/lib/graphics/light.cc

@@ -244 +244 @@
     if (this->lights[i])
       delete lights[i];
-  delete lights;
+  delete[] lights;
   LightManager::singletonRef = NULL;
 }

trunk/src/lib/graphics/render2D/element_2d.cc

@@ -90 +90 @@
   this->parent = NULL;
   this->children = new tList<Element2D>;
+  this->absDirection = 0.0;
   this->relDirection = 0.0;
   this->bRelCoorChanged = true;
@@ -844 +845 @@
 NullElement2D::~NullElement2D ()
 {
-  //delete singletonRef;
   NullElement2D::singletonRef = NULL;
 }

trunk/src/lib/graphics/render2D/element_2d.h

@@ -193 +193 @@
     Vector                  absCoordinate;      //!< absolute coordinates in the world ( from (0,0,0) )
     float                   relDirection;       //!< direction relative to the parent
-    float                   absDirection;       //!< absolute direvtion in the world ( from (0,0,1) )
+    float                   absDirection;       //!< absolute diretion in the world ( from (0,0,1) )
 
     Vector                  prevRelCoordinate;  //!< The last Relative Coordinate from the last update-Cycle.

trunk/src/lib/math/curve.cc

@@ -1 @@
-/* 
+/*
    orxonox - the future of 3D-vertical-scrollers
 
@@ -13 +13 @@
    co-programmer: Patrick Boenzli
 
-   ADD: Patrick Boenzli           B-Spline 
-
-
-   TODO: 
+   ADD: Patrick Boenzli           B-Spline
+
+
+   TODO:
      local-Time implementation
      NURBS
      tList implementation
-     
+
 */
 
@@ -33 +33 @@
 
 
-/** 
+/**
   *  default constructor for a Curve
 */
@@ -71 +71 @@
 void Curve::addNode(const Vector& newNode, unsigned int insertPosition)
 {
-
   if (this->nodeCount == 0 || insertPosition > this->nodeCount)
     return addNode(newNode);
@@ -85 +84 @@
     {
       while (tmpNode->next->number != insertPosition)
-        tmpNode= tmpNode->next;
+        tmpNode= tmpNode->next;
       insNode->next = tmpNode->next;
       tmpNode->next = insNode;
@@ -199 +198 @@
       tmpNode->factor = binCoef;
       if (tmpNode =tmpNode->next)
-        {
-          binCoef *=(n-k)/(k+1);
-          ++k;
-        }
+        {
+          binCoef *=(n-k)/(k+1);
+          ++k;
+        }
     }
 
@@ -212 +211 @@
       dirCurve = new BezierCurve(1);
       while(tmpNode->next)
-        {
-          Vector tmpVector = (tmpNode->next->position)- (tmpNode->position);
-          tmpVector.x*=(float)nodeCount;
-          tmpVector.y*=(float)nodeCount;
-          tmpVector.z*=(float)nodeCount;
-          tmpVector.normalize();
-          this->dirCurve->addNode(tmpVector);
-          tmpNode = tmpNode->next;
-        }
+        {
+          Vector tmpVector = (tmpNode->next->position)- (tmpNode->position);
+          tmpVector.x*=(float)nodeCount;
+          tmpVector.y*=(float)nodeCount;
+          tmpVector.z*=(float)nodeCount;
+          tmpVector.normalize();
+          this->dirCurve->addNode(tmpVector);
+          tmpNode = tmpNode->next;
+        }
     }
 }
@@ -229 +228 @@
  * @return the Position on the Path
 */
-Vector BezierCurve::calcPos(float t) 
+Vector BezierCurve::calcPos(float t)
 {
   Vector ret = Vector(0.0,0.0,0.0);
@@ -237 +236 @@
       double factor = pow(1.0-t,nodeCount-1);
       while(tmpNode)
-        {
-          ret.x += tmpNode->factor * factor * tmpNode->position.x;
-          ret.y += tmpNode->factor * factor * tmpNode->position.y;
-          ret.z += tmpNode->factor * factor * tmpNode->position.z;
-          factor *= t/(1.0-t); // same as pow but much faster.
-          
-          tmpNode = tmpNode->next;
-        }
+        {
+          ret.x += tmpNode->factor * factor * tmpNode->position.x;
+          ret.y += tmpNode->factor * factor * tmpNode->position.y;
+          ret.z += tmpNode->factor * factor * tmpNode->position.z;
+          factor *= t/(1.0-t); // same as pow but much faster.
+
+          tmpNode = tmpNode->next;
+        }
     }
   else if (nodeCount == 2)

trunk/src/lib/physics/physics_engine.cc

@@ -58 +58 @@
   }
   delete itPC;
-   delete this->connections;
+  delete this->connections;
 //
 //   // delete all PhysicsInterfaces, still in existence (this could be dangerous)

trunk/src/lib/util/list.h

@@ -82 +82 @@
     {
       listElement<T>* le = this->currentEl->next;
-      //delete this->currentEl->curr;
+      //delete this->currentEl->curr;  //! THIS IS EXTREMELY UNSAFE (the list only stores pointers not instances) //
       delete this->currentEl;
       this->currentEl = le;