Changeset 10917 for code/branches/cpp11_v2

Timestamp:

Dec 2, 2015, 11:32:08 PM (8 years ago)

Author:

landauf

Message:

made mapEntry const& wherever possible

Location:

code/branches/cpp11_v2/src

Files:

• libraries/core/Language.cc (modified) (1 diff)
• libraries/core/Loader.cc (modified) (1 diff)
• libraries/core/NamespaceNode.cc (modified) (1 diff)
• libraries/core/class/Identifier.cc (modified) (1 diff)
• libraries/core/input/InputManager.cc (modified) (1 diff)
• libraries/core/module/DynLibManager.cc (modified) (1 diff)
• libraries/core/module/ModuleInstance.cc (modified) (1 diff)
• libraries/core/module/PluginManager.cc (modified) (1 diff)
• modules/gametypes/SpaceRaceController.cc (modified) (1 diff)
• modules/gametypes/SpaceRaceManager.cc (modified) (1 diff)
• modules/mini4dgame/Mini4Dgame.cc (modified) (1 diff)
• modules/notifications/NotificationManager.cc (modified) (4 diffs)
• modules/notifications/NotificationQueue.cc (modified) (1 diff)
• modules/overlays/hud/HUDNavigation.cc (modified) (3 diffs)
• modules/overlays/hud/HUDRadar.cc (modified) (1 diff)
• modules/pickup/PickupManager.cc (modified) (1 diff)
• modules/pong/Pong.cc (modified) (1 diff)
• modules/questsystem/QuestManager.cc (modified) (2 diffs)
• modules/tetris/Tetris.cc (modified) (1 diff)
• orxonox/collisionshapes/CompoundCollisionShape.cc (modified) (2 diffs)
• orxonox/controllers/ArtificialController.cc (modified) (1 diff)
• orxonox/gametypes/Dynamicmatch.cc (modified) (8 diffs)
• orxonox/gametypes/Gametype.cc (modified) (7 diffs)
• orxonox/gametypes/LastManStanding.cc (modified) (3 diffs)
• orxonox/gametypes/LastTeamStanding.cc (modified) (3 diffs)
• orxonox/gametypes/TeamBaseMatch.cc (modified) (1 diff)
• orxonox/gametypes/TeamDeathmatch.cc (modified) (1 diff)
• orxonox/gametypes/TeamGametype.cc (modified) (6 diffs)
• orxonox/gametypes/UnderAttack.cc (modified) (2 diffs)
• orxonox/weaponsystem/Munition.cc (modified) (2 diffs)
• orxonox/weaponsystem/Weapon.cc (modified) (3 diffs)
• orxonox/weaponsystem/WeaponSet.cc (modified) (2 diffs)
• orxonox/weaponsystem/WeaponSystem.cc (modified) (4 diffs)

code/branches/cpp11_v2/src/libraries/core/Language.cc

@@ -107 +107 @@
     Language::~Language()
     {
-        for (auto& mapEntry : this->languageEntries_)
+        for (const auto& mapEntry : this->languageEntries_)
             delete (mapEntry.second);
     }

code/branches/cpp11_v2/src/libraries/core/Loader.cc

@@ -261 +261 @@
         {
             bool expectedValue = true;
-            for (auto& mapEntry : luaTags)
+            for (const auto& mapEntry : luaTags)
             {
                 if (mapEntry.second == expectedValue)

code/branches/cpp11_v2/src/libraries/core/NamespaceNode.cc

@@ -103 +103 @@
                 bool bFoundMatchingNamespace = false;
 
-                for (auto& mapEntry : this->subnodes_)
+                for (const auto& mapEntry : this->subnodes_)
                 {
                     if (mapEntry.first.find(firstPart) == (mapEntry.first.size() - firstPart.size()))

code/branches/cpp11_v2/src/libraries/core/class/Identifier.cc

@@ -92 +92 @@
             const_cast<Identifier*>(directChild)->directParents_.remove(this);
 
-        for (auto& mapEntry : this->configValues_)
+        for (const auto& mapEntry : this->configValues_)
             delete (mapEntry.second);
-        for (auto& mapEntry : this->xmlportParamContainers_)
+        for (const auto& mapEntry : this->xmlportParamContainers_)
             delete (mapEntry.second);
-        for (auto& mapEntry : this->xmlportObjectContainers_)
+        for (const auto& mapEntry : this->xmlportObjectContainers_)
             delete (mapEntry.second);
     }

code/branches/cpp11_v2/src/libraries/core/input/InputManager.cc

@@ -373 +373 @@
         // check whether a state has changed its EMPTY situation
         bool bUpdateRequired = false;
-        for (auto& mapEntry : activeStates_)
+        for (const auto& mapEntry : activeStates_)
         {
             if (mapEntry.second->hasExpired())

code/branches/cpp11_v2/src/libraries/core/module/DynLibManager.cc

@@ -75 +75 @@
     {
         // Unload & delete resources in turn
-        for (auto& mapEntry : mLibList)
+        for (const auto& mapEntry : mLibList)
         {
             mapEntry.second->unload();

code/branches/cpp11_v2/src/libraries/core/module/ModuleInstance.cc

@@ -75 +75 @@
         std::map<StaticInitialization::Type, std::set<StaticallyInitializedInstance*>> copy(this->staticallyInitializedInstancesByType_);
         this->staticallyInitializedInstancesByType_.clear();
-        for (auto& mapEntry : copy)
+        for (const auto& mapEntry : copy)
             for (std::set<StaticallyInitializedInstance*>::iterator it2 = mapEntry.second.begin(); it2 != mapEntry.second.end(); ++it2)
                 delete (*it2);

code/branches/cpp11_v2/src/libraries/core/module/PluginManager.cc

@@ -59 +59 @@
         ModifyConsoleCommand("PluginManager", __CC_PluginManager_unload_name).setObject(nullptr);
 
-        for (auto& mapEntry : this->references_)
+        for (const auto& mapEntry : this->references_)
             delete mapEntry.second;
-        for (auto& mapEntry : this->plugins_)
+        for (const auto& mapEntry : this->plugins_)
             delete mapEntry.second;
     }

code/branches/cpp11_v2/src/modules/gametypes/SpaceRaceController.cc

@@ -161 +161 @@
 
         std::vector<RaceCheckPoint*> returnVec;
-        for (auto& mapEntry : zaehler)
+        for (const auto& mapEntry : zaehler)
         {
             if (mapEntry.second == maxWays)

code/branches/cpp11_v2/src/modules/gametypes/SpaceRaceManager.cc

@@ -77 +77 @@
         }
 
-        for (auto& mapEntry : players_)
+        for (const auto& mapEntry : players_)
         {
 

code/branches/cpp11_v2/src/modules/mini4dgame/Mini4Dgame.cc

@@ -155 +155 @@
     {
         // first spawn human players to assign always the left bat to the player in singleplayer
-        for (auto& mapEntry : this->players_)
+        for (const auto& mapEntry : this->players_)
             if (mapEntry.first->isHumanPlayer() && (mapEntry.first->isReadyToSpawn() || this->bForceSpawn_))
                 this->spawnPlayer(mapEntry.first);
         // now spawn bots
-        for (auto& mapEntry : this->players_)
+        for (const auto& mapEntry : this->players_)
             if (!mapEntry.first->isHumanPlayer() && (mapEntry.first->isReadyToSpawn() || this->bForceSpawn_))
                 this->spawnPlayer(mapEntry.first);

code/branches/cpp11_v2/src/modules/notifications/NotificationManager.cc

@@ -69 +69 @@
     {
         // Destroys all Notifications.
-        for(auto& mapEntry : this->allNotificationsList_)
+        for(const auto& mapEntry : this->allNotificationsList_)
             mapEntry.second->destroy();
         this->allNotificationsList_.clear();
@@ -152 +152 @@
         bool executed = false;
         // Clear all NotificationQueues that have the input sender as target.
-        for(auto& mapEntry : this->queues_) // Iterate through all NotificationQueues.
+        for(const auto& mapEntry : this->queues_) // Iterate through all NotificationQueues.
         {
             const std::set<std::string>& set = mapEntry.second->getTargetsSet();
@@ -187 +187 @@
 
         // Insert the Notification in all NotificationQueues that have its sender as target.
-        for(auto& mapEntry : this->queues_) // Iterate through all NotificationQueues.
+        for(const auto& mapEntry : this->queues_) // Iterate through all NotificationQueues.
         {
             const std::set<std::string>& set = mapEntry.second->getTargetsSet();
@@ -345 +345 @@
 
         // Iterate through all Notifications to determine whether any of them should belong to the newly registered NotificationQueue.
-        for(auto& mapEntry : this->allNotificationsList_)
+        for(const auto& mapEntry : this->allNotificationsList_)
         {
             if(!bAll && set.find(mapEntry.second->getSender()) != set.end()) // Checks whether the listener has the sender of the current Notification as target.

code/branches/cpp11_v2/src/modules/notifications/NotificationQueue.cc

@@ -206 +206 @@
         {
             // Add all Notifications that have been created after this NotificationQueue was created.
-            for(auto& mapEntry : *notifications)
+            for(const auto& mapEntry : *notifications)
             {
                 if(mapEntry.first >= this->creationTime_)

code/branches/cpp11_v2/src/modules/overlays/hud/HUDNavigation.cc

@@ -131 +131 @@
         }
         this->fontName_ = font;
-        for (auto& mapEntry : this->activeObjectList_)
+        for (const auto& mapEntry : this->activeObjectList_)
         {
             if (mapEntry.second.text_ != nullptr)
@@ -151 +151 @@
         }
         this->textSize_ = size;
-        for (auto& mapEntry : this->activeObjectList_)
+        for (const auto& mapEntry : this->activeObjectList_)
         {
             if (mapEntry.second.text_)
@@ -531 +531 @@
         float yScale = this->getActualSize().y;
 
-        for (auto& mapEntry : this->activeObjectList_)
+        for (const auto& mapEntry : this->activeObjectList_)
         {
             if (mapEntry.second.health_ != nullptr)

code/branches/cpp11_v2/src/modules/overlays/hud/HUDRadar.cc

@@ -92 +92 @@
             Ogre::OverlayManager::getSingleton().destroyOverlayElement(this->map3DBack_);
 
-            for (auto& mapEntry : this->radarObjects_)
+            for (const auto& mapEntry : this->radarObjects_)
             {
                 Ogre::OverlayManager::getSingleton().destroyOverlayElement(mapEntry.second);

code/branches/cpp11_v2/src/modules/pickup/PickupManager.cc

@@ -91 +91 @@
 
         // Destroying all the PickupInventoryContainers that are still there.
-        for(auto& mapEntry : this->pickupInventoryContainers_)
+        for(const auto& mapEntry : this->pickupInventoryContainers_)
             delete mapEntry.second;
         this->pickupInventoryContainers_.clear();

code/branches/cpp11_v2/src/modules/pong/Pong.cc

@@ -211 +211 @@
     {
         // first spawn human players to assign always the left bat to the player in singleplayer
-        for (auto& mapEntry : this->players_)
+        for (const auto& mapEntry : this->players_)
             if (mapEntry.first->isHumanPlayer() && (mapEntry.first->isReadyToSpawn() || this->bForceSpawn_))
                 this->spawnPlayer(mapEntry.first);
         // now spawn bots
-        for (auto& mapEntry : this->players_)
+        for (const auto& mapEntry : this->players_)
             if (!mapEntry.first->isHumanPlayer() && (mapEntry.first->isReadyToSpawn() || this->bForceSpawn_))
                 this->spawnPlayer(mapEntry.first);

code/branches/cpp11_v2/src/modules/questsystem/QuestManager.cc

@@ -235 +235 @@
     {
         int numQuests = 0;
-        for(auto& mapEntry : this->questMap_)
+        for(const auto& mapEntry : this->questMap_)
         {
             if(mapEntry.second->getParentQuest() == nullptr && !mapEntry.second->isInactive(player))
@@ -255 +255 @@
     Quest* QuestManager::getRootQuest(PlayerInfo* player, int index)
     {
-        for(auto& mapEntry : this->questMap_)
+        for(const auto& mapEntry : this->questMap_)
         {
             if(mapEntry.second->getParentQuest() == nullptr && !mapEntry.second->isInactive(player) && index-- == 0)

code/branches/cpp11_v2/src/modules/tetris/Tetris.cc

@@ -341 +341 @@
     {
         // Spawn a human player.
-        for (auto& mapEntry : this->players_)
+        for (const auto& mapEntry : this->players_)
             if (mapEntry.first->isHumanPlayer() && (mapEntry.first->isReadyToSpawn() || this->bForceSpawn_))
                 this->spawnPlayer(mapEntry.first);

code/branches/cpp11_v2/src/orxonox/collisionshapes/CompoundCollisionShape.cc

@@ -65 +65 @@
         {
             // Delete all children
-            for (auto& mapEntry : this->attachedShapes_)
+            for (const auto& mapEntry : this->attachedShapes_)
             {
                 // make sure that the child doesn't want to detach itself --> speedup because of the missing update
@@ -266 +266 @@
         std::vector<CollisionShape*> shapes;
         // Iterate through all attached CollisionShapes and add them to the list of shapes.
-        for(auto& mapEntry : this->attachedShapes_)
+        for(const auto& mapEntry : this->attachedShapes_)
             shapes.push_back(mapEntry.first);
 

code/branches/cpp11_v2/src/orxonox/controllers/ArtificialController.cc

@@ -231 +231 @@
     int ArtificialController::getFiremode(std::string name)
     {
-        for (auto& mapEntry : this->weaponModes_)
+        for (const auto& mapEntry : this->weaponModes_)
         {
             if (mapEntry.first == name)

code/branches/cpp11_v2/src/orxonox/gametypes/Dynamicmatch.cc

@@ -405 +405 @@
     void Dynamicmatch::rewardPig()
     {
-        for (auto& mapEntry : this->playerParty_) //durch alle Spieler iterieren und alle piggys finden
+        for (const auto& mapEntry : this->playerParty_) //durch alle Spieler iterieren und alle piggys finden
         {
             if (mapEntry.second==piggy)//Spieler mit der Pig-party frags++
@@ -441 +441 @@
             if (tutorial) // Announce selectionphase
             {
-             for (auto& mapEntry : this->playerParty_)
+             for (const auto& mapEntry : this->playerParty_)
                 {
                     if (!mapEntry.first)//in order to catch nullpointer
@@ -456 +456 @@
              if(tutorial&&(!notEnoughKillers)&&(!notEnoughChasers)) //Selection phase over
              {
-                  for (auto& mapEntry : this->playerParty_)
+                  for (const auto& mapEntry : this->playerParty_)
                   {
                        if (!mapEntry.first)//in order to catch nullpointer
@@ -490 +490 @@
             if (tutorial) // Announce selectionphase
             {
-             for (auto& mapEntry : this->playerParty_)
+             for (const auto& mapEntry : this->playerParty_)
                 {
                     if (!mapEntry.first)//in order to catch nullpointer
@@ -505 +505 @@
             if(tutorial&&(!notEnoughPigs)&&(!notEnoughChasers)) //Selection phase over
              {
-                  for (auto& mapEntry : this->playerParty_)
+                  for (const auto& mapEntry : this->playerParty_)
                   {
                        if (!mapEntry.first)
@@ -540 +540 @@
             if (tutorial) // Announce selectionphase
             {
-             for (auto& mapEntry : this->playerParty_)
+             for (const auto& mapEntry : this->playerParty_)
                 {
                     if (!mapEntry.first)//in order to catch nullpointer
@@ -555 +555 @@
              if(tutorial&&(!notEnoughPigs)&&(!notEnoughKillers)) //Selection phase over
              {
-                  for (auto& mapEntry : this->playerParty_)
+                  for (const auto& mapEntry : this->playerParty_)
                   {
                        if (!mapEntry.first)
@@ -620 +620 @@
         else if(tutorial) // Announce selectionphase
         {
-            for (auto& mapEntry : this->playerParty_)
+            for (const auto& mapEntry : this->playerParty_)
             {
                 if (mapEntry.first->getClientID() == NETWORK_PEER_ID_UNKNOWN)

code/branches/cpp11_v2/src/orxonox/gametypes/Gametype.cc

@@ -139 +139 @@
         if (!this->gtinfo_->hasStarted())
         {
-            for (auto& mapEntry : this->players_)
+            for (const auto& mapEntry : this->players_)
             {
                 // Inform the GametypeInfo that the player is ready to spawn.
@@ -169 +169 @@
         }
 
-        for (auto& mapEntry : this->players_)
+        for (const auto& mapEntry : this->players_)
         {
             if (mapEntry.first->getControllableEntity())
@@ -404 +404 @@
                     bool allplayersready = true;
                     bool hashumanplayers = false;
-                    for (auto& mapEntry : this->players_)
+                    for (const auto& mapEntry : this->players_)
                     {
                         if (!mapEntry.first->isReadyToSpawn())
@@ -430 +430 @@
     void Gametype::spawnPlayersIfRequested()
     {
-        for (auto& mapEntry : this->players_)
+        for (const auto& mapEntry : this->players_)
         {
             if (mapEntry.first->isReadyToSpawn() || this->bForceSpawn_)
@@ -439 +439 @@
     void Gametype::spawnDeadPlayersIfRequested()
     {
-        for (auto& mapEntry : this->players_)
+        for (const auto& mapEntry : this->players_)
             if (mapEntry.second.state_ == PlayerState::Dead)
                 if (mapEntry.first->isReadyToSpawn() || this->bForceSpawn_)
@@ -538 +538 @@
     GSLevelMementoState* Gametype::exportMementoState()
     {
-        for (auto& mapEntry : this->players_)
+        for (const auto& mapEntry : this->players_)
         {
             if (mapEntry.first->isHumanPlayer() && mapEntry.first->getControllableEntity() && mapEntry.first->getControllableEntity()->getCamera())
@@ -587 +587 @@
 
         // find correct player and assign default entity with original position & orientation
-        for (auto& mapEntry : this->players_)
+        for (const auto& mapEntry : this->players_)
         {
             if (mapEntry.first->isHumanPlayer())

code/branches/cpp11_v2/src/orxonox/gametypes/LastManStanding.cc

@@ -56 +56 @@
     void LastManStanding::spawnDeadPlayersIfRequested()
     {
-        for (auto& mapEntry : this->players_)
+        for (const auto& mapEntry : this->players_)
             if (mapEntry.second.state_ == PlayerState::Dead)
             {
@@ -114 +114 @@
     {
         int min=lives;
-        for (auto& mapEntry : this->playerLives_)
+        for (const auto& mapEntry : this->playerLives_)
         {
             if (mapEntry.second<=0)
@@ -128 +128 @@
         Gametype::end();
 
-        for (auto& mapEntry : this->playerLives_)
+        for (const auto& mapEntry : this->playerLives_)
         {
             if (mapEntry.first->getClientID() == NETWORK_PEER_ID_UNKNOWN)

code/branches/cpp11_v2/src/orxonox/gametypes/LastTeamStanding.cc

@@ -145 +145 @@
     void LastTeamStanding::spawnDeadPlayersIfRequested()
     {
-        for (auto& mapEntry : this->players_)
+        for (const auto& mapEntry : this->players_)
             if (mapEntry.second.state_ == PlayerState::Dead)
             {
@@ -229 +229 @@
         int party = -1;
         //find a player who survived
-        for (auto& mapEntry : this->playerLives_)
+        for (const auto& mapEntry : this->playerLives_)
         {
           if (mapEntry.first->getClientID() == NETWORK_PEER_ID_UNKNOWN)
@@ -255 +255 @@
     {
         int min = lives;
-        for (auto& mapEntry : this->playerLives_)
+        for (const auto& mapEntry : this->playerLives_)
         {
             if (mapEntry.second <= 0)

code/branches/cpp11_v2/src/orxonox/gametypes/TeamBaseMatch.cc

@@ -187 +187 @@
             }
 
-            for (auto& mapEntry : this->teamnumbers_)
+            for (const auto& mapEntry : this->teamnumbers_)
             {
                 if (mapEntry.first->getClientID() == NETWORK_PEER_ID_UNKNOWN)

code/branches/cpp11_v2/src/orxonox/gametypes/TeamDeathmatch.cc

@@ -69 +69 @@
         int winnerTeam = 0;
         int highestScore = 0;
-        for (auto& mapEntry : this->players_)
+        for (const auto& mapEntry : this->players_)
         {
             if ( this->getTeamScore(mapEntry.first) > highestScore )

code/branches/cpp11_v2/src/orxonox/gametypes/TeamGametype.cc

@@ -99 +99 @@
         std::vector<unsigned int> playersperteam(this->teams_, 0);
 
-        for (auto& mapEntry : this->teamnumbers_)
+        for (const auto& mapEntry : this->teamnumbers_)
             if (mapEntry.second < static_cast<int>(this->teams_) && mapEntry.second >= 0)
                 playersperteam[mapEntry.second]++;
@@ -141 +141 @@
     void TeamGametype::spawnDeadPlayersIfRequested()
     {
-        for (auto& mapEntry : this->players_)\
+        for (const auto& mapEntry : this->players_)\
         {
             if(allowedInGame_[mapEntry.first] == false)//check if dead player is allowed to enter
@@ -178 +178 @@
         if(!player || this->getTeam(player) == -1)
             return 0;
-        for (auto& mapEntry : this->players_)
+        for (const auto& mapEntry : this->players_)
         {
             if ( this->getTeam(mapEntry.first) ==  this->getTeam(player) )
@@ -191 +191 @@
     {
         int teamSize = 0;
-        for (auto& mapEntry : this->teamnumbers_)
+        for (const auto& mapEntry : this->teamnumbers_)
         {
             if (mapEntry.second == team)
@@ -202 +202 @@
     {
         int teamSize = 0;
-        for (auto& mapEntry : this->teamnumbers_)
+        for (const auto& mapEntry : this->teamnumbers_)
         {
             if (mapEntry.second == team  && mapEntry.first->isHumanPlayer())
@@ -376 +376 @@
     void TeamGametype::announceTeamWin(int winnerTeam)
     {
-        for (auto& mapEntry : this->teamnumbers_)
+        for (const auto& mapEntry : this->teamnumbers_)
         {
             if (mapEntry.first->getClientID() == NETWORK_PEER_ID_UNKNOWN)

code/branches/cpp11_v2/src/orxonox/gametypes/UnderAttack.cc

@@ -74 +74 @@
         this->gameEnded_ = true;
 
-        for (auto& mapEntry : this->teamnumbers_)
+        for (const auto& mapEntry : this->teamnumbers_)
         {
             if (mapEntry.first->getClientID() == NETWORK_PEER_ID_UNKNOWN)
@@ -155 +155 @@
                 ChatManager::message(message);
 
-                for (auto& mapEntry : this->teamnumbers_)
+                for (const auto& mapEntry : this->teamnumbers_)
                 {
                     if (mapEntry.first->getClientID() == NETWORK_PEER_ID_UNKNOWN)

code/branches/cpp11_v2/src/orxonox/weaponsystem/Munition.cc

@@ -55 +55 @@
     Munition::~Munition()
     {
-        for (auto& mapEntry : this->currentMagazines_)
+        for (const auto& mapEntry : this->currentMagazines_)
             delete mapEntry.second;
     }
@@ -316 +316 @@
             {
                 bool change = false;
-                for (auto& mapEntry : this->currentMagazines_)
+                for (const auto& mapEntry : this->currentMagazines_)
                 {
                     // Add munition if the magazine isn't full (but only to loaded magazines)

code/branches/cpp11_v2/src/orxonox/weaponsystem/Weapon.cc

@@ -61 +61 @@
                 this->weaponPack_->removeWeapon(this);
 
-            for (auto& mapEntry : this->weaponmodes_)
+            for (const auto& mapEntry : this->weaponmodes_)
                 mapEntry.second->destroy();
         }
@@ -136 +136 @@
     void Weapon::reload()
     {
-        for (auto& mapEntry : this->weaponmodes_)
+        for (const auto& mapEntry : this->weaponmodes_)
             mapEntry.second->reload();
     }
@@ -148 +148 @@
     void Weapon::notifyWeaponModes()
     {
-        for (auto& mapEntry : this->weaponmodes_)
+        for (const auto& mapEntry : this->weaponmodes_)
             mapEntry.second->setWeapon(this);
     }

code/branches/cpp11_v2/src/orxonox/weaponsystem/WeaponSet.cc

@@ -63 +63 @@
     {
         // Fire all WeaponPacks with their defined weaponmode
-        for (auto& mapEntry : this->weaponpacks_)
+        for (const auto& mapEntry : this->weaponpacks_)
             if (mapEntry.second != WeaponSystem::WEAPON_MODE_UNASSIGNED)
                 mapEntry.first->fire(mapEntry.second);
@@ -71 +71 @@
     {
         // Reload all WeaponPacks with their defined weaponmode
-        for (auto& mapEntry : this->weaponpacks_)
+        for (const auto& mapEntry : this->weaponpacks_)
             mapEntry.first->reload();
     }

code/branches/cpp11_v2/src/orxonox/weaponsystem/WeaponSystem.cc

@@ -196 +196 @@
 
         // Assign the desired weaponmode to the firemodes
-        for (auto& mapEntry : this->weaponSets_)
+        for (const auto& mapEntry : this->weaponSets_)
         {
             unsigned int weaponmode = wPack->getDesiredWeaponmode(mapEntry.first);
@@ -219 +219 @@
 
         // Remove all added links from the WeaponSets
-        for (auto& mapEntry : this->weaponSets_)
+        for (const auto& mapEntry : this->weaponSets_)
             mapEntry.second->removeWeaponmodeLink(wPack);
 
@@ -268 +268 @@
         // Check if the WeaponSet belongs to this WeaponSystem
         bool foundWeaponSet = false;
-        for (auto& mapEntry : this->weaponSets_)
+        for (const auto& mapEntry : this->weaponSets_)
         {
             if (mapEntry.second == wSet)
@@ -296 +296 @@
     void WeaponSystem::reload()
     {
-        for (auto& mapEntry : this->weaponSets_)
+        for (const auto& mapEntry : this->weaponSets_)
             mapEntry.second->reload();
     }