Changeset 11990 for code/branches/Masterserver_FS18/src/libraries/network/GamestateManager.cc

Timestamp:

May 24, 2018, 3:24:00 PM (6 years ago)

Author:

mdedial

Message:

Clean up code, remove TODOs and dead code.

File:

• code/branches/Masterserver_FS18/src/libraries/network/GamestateManager.cc (modified) (4 diffs)

code/branches/Masterserver_FS18/src/libraries/network/GamestateManager.cc

@@ -86 +86 @@
   bool GamestateManager::update()
   {
-    return getSnapshot();
+    return this->getSnapshot();
   }
 
@@ -119 +119 @@
     for(const auto& gsPair : this->gamestateQueue)
     {
-      OrxVerify(processGamestate(gsPair.second), "ERROR: could not process Gamestate");
-      sendAck(gsPair.second->getID(), gsPair.second->getPeerID());
+      OrxVerify(this->processGamestate(gsPair.second), "ERROR: could not process Gamestate");
+      this->sendAck(gsPair.second->getID(), gsPair.second->getPeerID());
       delete gsPair.second;
     }
@@ -214 +214 @@
 
     std::vector<packet::Gamestate*> peerGamestates;
-    // TODO: mapEntry is an incredibly bad name
     for(const auto& mapEntry : this->peerMap_)
     {
@@ -256 +255 @@
 
     // Create a copy of the gamestate
-    // TODO: Does this actually create a copy of the gamestate?
     packet::Gamestate *gs = new packet::Gamestate(*gamestate); //this is neccessary because the gamestate are being kept (to diff them later on) for each client seperately
     this->peerMap_[peerID].gamestates[gamestate->getID()] = gs;