Changeset 9621 for code/branches/formationupdate/src/orxonox/worldentities/pawns

Timestamp:

May 7, 2013, 3:58:33 PM (12 years ago)

Author:

maxima

Message:

 

File:

• code/branches/formationupdate/src/orxonox/worldentities/pawns/Pawn.cc (modified) (6 diffs)

code/branches/formationupdate/src/orxonox/worldentities/pawns/Pawn.cc

@@ -309 +309 @@
 
 
-    /* Two functions to find the slaves of a Pawn
-     *
-     */
+    // A function to check if this pawn's controller is the master of any formationcontroller
     bool Pawn::hasSlaves()
     {
@@ -318 +316 @@
                          it != ObjectList<FormationController>::end(); ++it )
                 {
-                        // checks if the Pawn has a slave
+                        // checks if the pawn's controller has a slave
                         if (this->hasHumanController() && it->getMaster() == this->getPlayer()->getController())
                                 return true;
@@ -325 +323 @@
     }
 
+    // A function that returns a slave of the pawn's controller
     Controller* Pawn::getSlave(){
         for (ObjectList<FormationController>::iterator it =
@@ -331 +330 @@
         {
                 if (this->hasHumanController() && it->getMaster() == this->getPlayer()->getController())
-                        return it->getThis();
+                        return it->getController();
         }
         return 0;
@@ -353 +352 @@
             if (this->getPlayer() && this->getPlayer()->getControllableEntity() == this)
             {
-
-                // Do different things if Pawn is the Master of a Formation
+                // Start to control a new entity if you're the master of a formation
                 if(this->hasSlaves())
                 {
@@ -363 +361 @@
                         slave->setControllableEntity(0);
 
-
-                        // set new Master
+                        // set a new master within the formation
                                         orxonox_cast<FormationController*>(this->getController())->setNewMasterWithinFormation(orxonox_cast<FormationController*>(slave));
-                        //orxonox_cast<FormationController*>(this->getController())->setNewMasterWithinFormation();
-
-
-                        /* TO DO: - setNewMasterWithinFormation() with an argument.
-                         *                - set slave as the new master within the formation
-                         *
-                         */
 
                                         // start to control a slave
                                 this->getPlayer()->startControl(entity);
-
-
                 }
                 else