Changeset 7126 in orxonox.OLD for trunk

Timestamp:

Feb 13, 2006, 1:14:53 PM (19 years ago)

Author:

bensch

Message:

orxonox/trunk: the delete Mechanism of PNode seems to work again.
This is done by always deleting the first node, and not walking through a list, that is eleminated during deletion time

Location:

trunk/src

Files:

• lib/coord/p_node.cc (modified) (4 diffs)
• orxonox.cc (modified) (1 diff)
• story_entities/game_world_data.cc (modified) (2 diffs)
• world_entities/environments/water.cc (modified) (1 diff)
• world_entities/space_ships/turbine_hover.cc (modified) (1 diff)
• world_entities/weapons/weapon.cc (modified) (2 diffs)

trunk/src/lib/coord/p_node.cc

@@ -80 +80 @@
 PNode::~PNode ()
 {
-  printf("delete PNode: %s::%s\n", this->getClassName(), this->getName());
+  PRINTF(4)("delete %s::%s\n", this->getClassName(), this->getName());
   // remove the Node, delete it's children (if required).
   std::list<PNode*>::iterator tmp = this->children.begin();
   std::list<PNode*>::iterator deleteNode;
+  unsigned int size;
   while(!this->children.empty())
-    while (tmp != this->children.end())
+  {
+    tmp = this->children.begin();
+    //while (tmp != this->children.end())
     {
       deleteNode = tmp;
       tmp++;
-      //      printf("TEST::%s(%s) %s\n", (*deleteNode)->getName(), (*deleteNode)->getClassName(), this->getName());
+      size = this->children.size();
       if ((this->parentMode & PNODE_PROHIBIT_CHILD_DELETE) ||
           ((*deleteNode)->parentMode & PNODE_PROHIBIT_DELETE_WITH_PARENT))
@@ -95 +98 @@
         if (this == PNode::nullParent && (*deleteNode)->parentMode & PNODE_REPARENT_TO_NULL)
         {
-          printf("%s::%s deletes PNode: %s::%s\n",
+          PRINTF(4)("%s::%s deletes %s::%s\n",
                  this->getClassName(), this->getName(),
                  (*deleteNode)->getClassName(), (*deleteNode)->getName());
@@ -102 +105 @@
         else
         {
-          printf("%s::%s reparents PNode: %s::%s\n",
+          PRINTF(4)("%s::%s reparents %s::%s\n",
                  this->getClassName(), this->getName(),
                  (*deleteNode)->getClassName(), (*deleteNode)->getName());
-
           (*deleteNode)->reparent();
         }
@@ -111 +113 @@
       else
       {
-        printf("%s::%s deletes PNode: %s::%s\n",
+        PRINTF(4)("%s::%s deletes PNode: %s::%s\n",
                this->getClassName(), this->getName(),
                (*deleteNode)->getClassName(), (*deleteNode)->getName());
         delete (*deleteNode);
       }
+      //if (size <= this->children.size()) break;
     }
+  }
 
   if (this->parent != NULL)

trunk/src/orxonox.cc

@@ -120 +120 @@
 
   SDL_QuitSubSystem(SDL_INIT_TIMER);
-  ClassList::debug(3);
+  ClassList::debug();
 
   PRINT(3)

trunk/src/story_entities/game_world_data.cc

@@ -252 +252 @@
   FastFactory::flushAll(true);
   GraphicsEngine::getInstance()->displayFPS(false);
-  State::setPlayer(NULL);
   // erease everything that is left.
   // delete PNode::getNullParent(); // not needed as this is also done in the next step (and also much cleaner)
@@ -261 +260 @@
     while (!nodeList->empty())
   {
-    ClassList::debug( 3, CL_PARENT_NODE);
-    PNode::getNullParent()->debugNode( 1);
-
-/*    printf("%s::%s\n", nodeList->front()->getClassName(), nodeList->front()->getName());*/
+//    ClassList::debug( 3, CL_PARENT_NODE);
+//    PNode::getNullParent()->debugNode(0);
+//    printf("%s::%s\n", nodeList->front()->getClassName(), nodeList->front()->getName());
     delete nodeList->front();
-    printf("OK\n");
   }
   /* remove the player object */
   if( this->localPlayer)
     delete this->localPlayer;
+  State::setPlayer(NULL);
 
   nodeList = ClassList::getList(CL_GRAPHICS_EFFECT);

trunk/src/world_entities/environments/water.cc

@@ -63 +63 @@
 Water::~Water()
 {
-  delete this->grid;
 }
 

trunk/src/world_entities/space_ships/turbine_hover.cc

@@ -80 +80 @@
   Weapon* wpLeft = new TestGun(1);
   wpLeft->setName("testGun Left");
-  Weapon* cannon = dynamic_cast<Weapon*>(Factory::fabricate(CL_HYPERBLASTER));
-
-  cannon->setName("BFG");
+  //Weapon* cannon = dynamic_cast<Weapon*>(Factory::fabricate(CL_HYPERBLASTER));
+
+  //  cannon->setName("BFG");
 
   this->addWeapon(wpLeft, 1, 0);
   this->addWeapon(wpRight,1 ,1);
-  this->addWeapon(cannon, 0, 2);
+  //this->addWeapon(cannon, 0, 2);
 
   this->getWeaponManager()->changeWeaponConfig(1);

trunk/src/world_entities/weapons/weapon.cc

@@ -29 +29 @@
 #include "state.h"
 #include "animation3d.h"
-#include "vector.h"
 
 #include "sound_source.h"
@@ -86 +85 @@
   this->soundSource = new SoundSource(this);       //< Every Weapon has exacty one SoundSource.
   this->emissionPoint.setParent(this);             //< One EmissionPoint, that is a PNode connected to the weapon. You can set this to the exitting point of the Projectiles
+  this->emissionPoint.setName("EmissionPoint");
+  this->emissionPoint.addNodeFlags(PNODE_PROHIBIT_DELETE_WITH_PARENT);
 
   this->defaultTarget = NULL;                      //< Nothing is Targeted by default.