Server: Use GetTickCount() to track time passed, instead of a timestamp

Author: provigz

InternetGamesServer/Util.hpp

@@ -212,31 +212,31 @@ class ChangeTimeTracker final
 public:
 	ChangeTimeTracker() :
 		val(),
-		timeLastChange(std::time(nullptr))
+		timeLastChange(GetTickCount())
 	{}
 	ChangeTimeTracker(T defaultVal) :
 		val(std::move(defaultVal)),
-		timeLastChange(std::time(nullptr))
+		timeLastChange(GetTickCount())
 	{}
 
 	inline const T& Get() const { return val; }
 	inline operator const T&() const { return val; }
 
-	inline std::time_t GetTimeLastChange() const { return timeLastChange; }
-	inline std::time_t GetSecondsSinceLastChange() const { return std::time(nullptr) - timeLastChange; }
+	inline DWORD GetTimeLastChange() const { return timeLastChange; }
+	inline DWORD GetTimePassedMS() const { return GetTickCount() - timeLastChange; }
 
 	void operator=(const T& otherVal)
 	{
 		if (val == otherVal)
 			return;
 
 		val = otherVal;
-		timeLastChange = std::time(nullptr);
+		timeLastChange = GetTickCount();
 	}
 
 private:
 	T val;
-	std::time_t timeLastChange;
+	DWORD timeLastChange;
 };
 
 /** Struct padding */

InternetGamesServer/Win7/Match.cpp

@@ -232,9 +232,9 @@ Match::Update()
 			if (m_endTime == 0)
 			{
 				SessionLog() << "[MATCH] " << m_guid << ": Game over, match will automatically close in 60 seconds!" << std::endl;
-				m_endTime = std::time(nullptr);
+				m_endTime = GetTickCount();
 			}
-			else if (std::time(nullptr) - m_endTime >= 60) // A minute has passed since the match ended
+			else if (GetTickCount() - m_endTime >= 60000) // A minute has passed since the match ended
 			{
 				SessionLog() << "[MATCH] " << m_guid << ": Match ended a minute ago, closing!" << std::endl;
 				m_state = STATE_ENDED;

InternetGamesServer/Win7/Match.hpp

@@ -101,7 +101,7 @@ class Match : public ::Match<PlayerSocket>
 	// Mutex to prevent simultaneous match processes, like adding/removing players and processing events and removal of the match
 	HANDLE m_mutex;
 
-	std::time_t m_endTime;
+	DWORD m_endTime;
 
 private:
 	Match(const Match&) = delete;

InternetGamesServer/Win7/PlayerSocket.cpp

@@ -72,7 +72,7 @@ PlayerSocket::ProcessMessages()
 		// Time out the client in states not involving participation in a match
 		if (m_state != STATE_WAITINGFOROPPONENTS &&
 			m_state != STATE_PLAYING &&
-			m_state.GetSecondsSinceLastChange() >= 60)
+			m_state.GetTimePassedMS() >= 60000)
 		{
 			throw std::runtime_error("Win7::PlayerSocket::ProcessMessages(): Timeout: Client has not switched from state "
 				+ std::to_string(m_state) + " for 60 seconds or more!");

InternetGamesServer/WinXP/Match.cpp

@@ -261,9 +261,9 @@ Match::Update()
 			if (m_endTime == 0)
 			{
 				SessionLog() << "[MATCH] " << m_guid << ": Game over, match will automatically close in 60 seconds!" << std::endl;
-				m_endTime = std::time(nullptr);
+				m_endTime = GetTickCount();
 			}
-			else if (std::time(nullptr) - m_endTime >= 60) // A minute has passed since the match ended
+			else if (GetTickCount() - m_endTime >= 60000) // A minute has passed since the match ended
 			{
 				SessionLog() << "[MATCH] " << m_guid << ": Match ended a minute ago, closing!" << std::endl;
 				m_state = STATE_ENDED;

InternetGamesServer/WinXP/Match.hpp

@@ -114,7 +114,7 @@ class Match : public ::Match<PlayerSocket>
 
 	std::array<uint32, MATCH_MAX_PLAYERS> m_playerComputerIDs; // IDs of computer players by seat.
 
-	std::time_t m_endTime;
+	DWORD m_endTime;
 
 private:
 	Match(const Match&) = delete;

InternetGamesServer/WinXP/PlayerSocket.cpp

@@ -175,7 +175,7 @@ PlayerSocket::ProcessMessages()
 		// Time out the client in states not involving participation in a match
 		if (m_state != STATE_WAITINGFOROPPONENTS &&
 			m_state != STATE_PLAYING &&
-			m_state.GetSecondsSinceLastChange() >= 60)
+			m_state.GetTimePassedMS() >= 60000)
 		{
 			throw std::runtime_error("WinXP::PlayerSocket::ProcessMessages(): Timeout: Client has not switched from state \""
 				+ StateToString(m_state) + "\" for 60 seconds or more!");