Changeset 8648 for code/branches/presentation/src/orxonox

Timestamp:

May 28, 2011, 8:53:14 PM (13 years ago)

Author:

scheusso

Message:

some network related fixes

Location:

code/branches/presentation/src/orxonox

Files:

• infos/GametypeInfo.cc (modified) (1 diff)
• items/Engine.cc (modified) (1 diff)
• worldentities/pawns/SpaceShip.cc (modified) (1 diff)
• worldentities/pawns/SpaceShip.h (modified) (1 diff)

code/branches/presentation/src/orxonox/infos/GametypeInfo.cc

@@ -342 +342 @@
         if(GameMode::isMaster())
         {
-            // Display "Press [Fire] to start the match" if the game has not yet ended.
-            if(!this->hasEnded())
-                NotificationListener::sendNotification("Press [Fire] to start the match", GametypeInfo::NOTIFICATION_SENDER, notificationMessageType::info, notificationSendMode::network, player->getClientID());
-            // Else display "Game has ended".
-            else
-                NotificationListener::sendNotification("Game has ended", GametypeInfo::NOTIFICATION_SENDER, notificationMessageType::info, notificationSendMode::network, player->getClientID());
+            if( player->isHumanPlayer() )
+            {
+                // Display "Press [Fire] to start the match" if the game has not yet ended.
+                if(!this->hasEnded())
+                    NotificationListener::sendNotification("Press [Fire] to start the match", GametypeInfo::NOTIFICATION_SENDER, notificationMessageType::info, notificationSendMode::network, player->getClientID());
+                // Else display "Game has ended".
+                else
+                    NotificationListener::sendNotification("Game has ended", GametypeInfo::NOTIFICATION_SENDER, notificationMessageType::info, notificationSendMode::network, player->getClientID());
+            }
         }
     }

code/branches/presentation/src/orxonox/items/Engine.cc

@@ -72 +72 @@
     Engine::~Engine()
     {
-
+        if( this->ship_ && this->ship_->hasEngine(this) )
+            this->ship_->removeEngine(this);
     }
 

code/branches/presentation/src/orxonox/worldentities/pawns/SpaceShip.cc

@@ -374 +374 @@
     void SpaceShip::removeAllEngines()
     {
-        for(unsigned int i=0; i<this->engineList_.size(); i++)
-            this->engineList_[i]->destroy();
+        while(this->engineList_.size())
+            this->engineList_.back()->destroy();
+    }
+    
+    void SpaceShip::removeEngine(Engine* engine)
+    {
+        for(std::vector<Engine*>::iterator it=this->engineList_.begin(); it!=this->engineList_.end(); ++it)
+        {
+            if(*it==engine)
+            {
+                this->engineList_.erase(it);
+                return;
+            }
+        }
     }
 

code/branches/presentation/src/orxonox/worldentities/pawns/SpaceShip.h

@@ -66 +66 @@
             inline const std::vector<Engine*>& getEngineList()
                 { return this->engineList_; }
+            void removeEngine(Engine* engine);
             void removeAllEngines();