Re-order comments a bit.

Author: slurmlord

Generals/Code/GameEngine/Source/GameNetwork/LANAPIhandlers.cpp

@@ -290,12 +290,12 @@ void LANAPI::handleRequestJoin( LANMessage *msg, UnsignedInt senderIP )
 #endif
 
 			// TheSuperHackers @bugfix slurmlord 18/09/2025 need to validate the name of the connecting player before
-			// allowing them to join to prevent messing up the format of game state string
+			// allowing them to join to prevent messing up the format of game state string. Commas, colons, semicolons etc.
+			// should not be in a player name. It should also not consist of only space characters.
 			constexpr WideChar IllegalNameChars[] = L",:;|\f\n\r\t\v";
 			if (canJoin && (wcscspn(msg->name, IllegalNameChars) || wcsspn(msg->name, L" ") == wcslen(msg->name)))
 			{
-				// Commas, colons, semicolons etc. should not be in a player name. It should also not consist of only space characters.
-				// If so, just deny with a duplicate name error (for backwards compatibility with retail)
+				// Just deny with a duplicate name reason, for backwards compatibility with retail
 				reply.LANMessageType = LANMessage::MSG_JOIN_DENY;
 				reply.GameNotJoined.reason = LANAPIInterface::RET_DUPLICATE_NAME;
 				reply.GameNotJoined.gameIP = m_localIP;

GeneralsMD/Code/GameEngine/Source/GameNetwork/LANAPIhandlers.cpp

@@ -291,12 +291,12 @@ void LANAPI::handleRequestJoin( LANMessage *msg, UnsignedInt senderIP )
 #endif
 
 			// TheSuperHackers @bugfix slurmlord 18/09/2025 need to validate the name of the connecting player before
-			// allowing them to join to prevent messing up the format of game state string
+			// allowing them to join to prevent messing up the format of game state string. Commas, colons, semicolons etc.
+			// should not be in a player name. It should also not consist of only space characters.
 			constexpr WideChar IllegalNameChars[] = L",:;|\f\n\r\t\v";
 			if (canJoin && (wcscspn(msg->name, IllegalNameChars) || wcsspn(msg->name, L" ") == wcslen(msg->name)))
 			{
-				// Commas, colons, semicolons etc. should not be in a player name. It should also not consist of only space characters.
-				// If so, just deny with a duplicate name error (for backwards compatibility with retail)
+				// Just deny with a duplicate name reason, for backwards compatibility with retail
 				reply.LANMessageType = LANMessage::MSG_JOIN_DENY;
 				reply.GameNotJoined.reason = LANAPIInterface::RET_DUPLICATE_NAME;
 				reply.GameNotJoined.gameIP = m_localIP;