[Scummvm-git-logs] scummvm master -> 107600267bdbde93066937b55a6c6c8a9e1abb5e

[LittleToonCat]

This automated email contains information about 1 new commit which have been
pushed to the 'scummvm' repo located at https://github.com/scummvm/scummvm .

Summary:
107600267b SCUMM HE: Add debug logging to checkPlayersLists


Commit: 107600267bdbde93066937b55a6c6c8a9e1abb5e
    https://github.com/scummvm/scummvm/commit/107600267bdbde93066937b55a6c6c8a9e1abb5e
Author: Little Cat (toontownlittlecat at gmail.com)
Date: 2023-10-16T00:52:08-03:00

Commit Message:
SCUMM HE: Add debug logging to checkPlayersLists

This is done to debug a bug that's only present in Windows' daily builds
, might revert this commit after the bug is identified and fixed.

Changed paths:
    engines/scumm/he/net/net_lobby.cpp


diff --git a/engines/scumm/he/net/net_lobby.cpp b/engines/scumm/he/net/net_lobby.cpp
index cafa9644977..5119943d74f 100644
--- a/engines/scumm/he/net/net_lobby.cpp
+++ b/engines/scumm/he/net/net_lobby.cpp
@@ -695,6 +695,8 @@ void Lobby::getPlayersList(int start, int end) {
 bool Lobby::_checkPlayersLists(Common::JSONArray other) {
 	// Check if the two players lists are different.
 	// This exists because (_playersList != other) doesn't work.
+	debugC(DEBUG_NETWORK, "LOBBY: _playerList.size() = %d", _playersList.size());
+	debugC(DEBUG_NETWORK, "LOBBY: other.size() = %d", other.size());
 	if (_playersList.size() != other.size())
 		return true;
 	for (uint i = 0; i < _playersList.size(); i++) {
@@ -702,9 +704,11 @@ bool Lobby::_checkPlayersLists(Common::JSONArray other) {
 		Common::JSONArray otherInfo = other[i]->asArray();
 
 		// Check if names are different.
+		debugC(DEBUG_NETWORK, "LOBBY: %s != %s", playerInfo[0]->asString().c_str(), otherInfo[0]->asString().c_str());
 		if (playerInfo[0]->asString() != otherInfo[0]->asString())
 			return true;
 		for (uint o = 1; o < 7; o++) {
+			debugC(DEBUG_NETWORK, "LOBBY: %lld != %lld", playerInfo[o]->asIntegerNumber(), otherInfo[o]->asIntegerNumber());
 			if (playerInfo[o]->asIntegerNumber() != otherInfo[o]->asIntegerNumber())
 				return true;
 		}