Changeset 8124 in orxonox.OLD for branches/cr/src/lib/collision_reaction

Timestamp:

Jun 3, 2006, 2:36:03 PM (18 years ago)

Author:

patrick

Message:

cr: some bugs found and solved, minor cleanup

Location:

branches/cr/src/lib/collision_reaction

Files:

• collision.cc (modified) (1 diff)
• collision.h (modified) (1 diff)
• collision_handle.cc (modified) (4 diffs)

branches/cr/src/lib/collision_reaction/collision.cc

@@ -66 +66 @@
   this->entityBCollide = false;
 
+  this->bDispatched = false;
+
   this->collisionEvents.clear();
 }

branches/cr/src/lib/collision_reaction/collision.h

@@ -2 +2 @@
  * @file collision.h
  *  Definition of a collision as a two WE hit each other
+ *
+ *  A is shared between two WorldEntity's CollisionHandles if both are subscribed to this event. In this case only one
+ *  of the two CollisionHandles will calculate the CollisionReaction and the bDispatched flag will be set afterwards
+ *  to signal that it's already cared about and should be ignored.
  */
 

branches/cr/src/lib/collision_reaction/collision_handle.cc

@@ -80 +80 @@
 /**
  * registers a new Collision Object
+ *  @param entityA WorldEntity A of the collision
+ *  @param entityB WorldEntity B of the collision
  * if a there is already a collision object with the same stats
  * registration will be skipped and the last collision object is returned
@@ -97 +99 @@
   // now register it as a shared collision with the other collision entity
   CollisionHandle* ch = entityB->getCollisionHandle(this->type);
-  if( ch != NULL) {
+  if( ch != NULL)
     ch->registerSharedCollision(c);
-    c->setEntityBCollide(true);
-  }
 
   return c;
@@ -134 +134 @@
   this->bDispatched = false;
 
-  // first element only
+  // checks if these WorldEntities have already collided or if its a new collision -> create a new Collision object
  Collision* c = this->registerCollision(collisionEvent->getEntityA(), collisionEvent->getEntityB());
  c->setEntityACollide(true);
@@ -178 +178 @@
   vector<long>::iterator it = this->targetList.begin();
   for(; it < this->targetList.end(); it++)
-  {
-    if( !collisionEvent->getEntityA()->isA((ClassID)(*it)))
-      return false;
-  }
+    if( collisionEvent->getEntityA()->isA((ClassID)(*it)))
+      return true;
 
-  return true;
+  return false;
 }