Fixed SortStats

stepmania/src/NetworkSyncServer.cpp

@@ -159,7 +159,7 @@ void StepManiaLanServer::ParseData(PacketFunctions& Packet, const unsigned int c
 	case NSCGSU:
 		// StatsUpdate
 		Client[clientNum]->UpdateStats(Packet);
-		if (Client[clientNum]->lowerJudge)
+		if (!Client[clientNum]->lowerJudge)
 			CheckLowerJudge(clientNum);
 		break;
 	case NSCSU:
@@ -390,32 +390,23 @@ void StepManiaLanServer::PopulatePlayersPtr(vector<LanPlayer*> &playersPtr) {
 int StepManiaLanServer::SortStats(vector<LanPlayer*> &playersPtr)
 {
 	LanPlayer *tmp;
-	StatsNameChange = false;	//Set it to no name so stats will not send if there is not a change
+	bool isChanged;
-	bool allZero = true;
 
 	PopulatePlayersPtr(playersPtr);
 
-	for (unsigned int x = 0; x < playersPtr.size(); ++x)
-	{
-		 //See if all the players have zero. Used to send names if everyone is 0
-		if ((playersPtr[x]->score > 0) && allZero)
-			allZero = false;
 
-		if ((x+1) < playersPtr.size())
+	do
+	{
+		isChanged = false;
+		for (int x = 0; x < int(playersPtr.size())-1; ++x)
 			if ((playersPtr[x]->score) < (playersPtr[x+1]->score))
 			{
 				tmp = playersPtr[x];
 				playersPtr[x] = playersPtr[x+1];
 				playersPtr[x+1] = tmp;
-				x = 0;
+				isChanged = true;
-				StatsNameChange = true;
 			}
-
+	} while (isChanged);
-	}
-	//If there are players all at 0, the name lsit won't be updated
-	//this make sure it is sent to the server eventually
-	if (!StatsNameChange && allZero)
-		StatsNameChange = true;
 
 	return playersPtr.size();
 }
@@ -426,11 +417,6 @@ void StepManiaLanServer::SendStatsToClients()
 
 	SortStats(playersPtr); //Return number of players
 
-	//If there was a change in the name data, send it to the clients.
-	//Used to save bandwidth and some processing time.
-	//Disabled this. FUTURE: Figure out why it dosn't work correctly and fix.
-//	if (StatsNameChange)
-//	{
 	/* Write and Send name packet */
 	Reply.ClearPacket();
 	Reply.Write1(NSCGSU + NSServerOffset);
@@ -443,7 +429,6 @@ void StepManiaLanServer::SendStatsToClients()
 		if (Client[x]->InGame)
 			SendNetPacket(x, Reply);
 
-//	}
 
 	/* Write and send Combo packet */
 	Reply.ClearPacket();

stepmania/src/NetworkSyncServer.h

@@ -90,7 +90,6 @@ private:
 	time_t statsTime;
 	GameInfo CurrentSongInfo;
 	GameInfo LastSongInfo;
-	bool StatsNameChange;
 	bool SecondSameSelect;
 	vector<CString> bannedIPs;