Commit part 3 of patch #901, score submission is now ported over to the new API.

git-svn-id: svn+ssh://svn.gna.org/svn/warzone/trunk@3220 4a71c877-e1ca-e34f-864e-861f7616d084

src/multiplay.c

@@ -700,7 +700,7 @@ BOOL recvMessage(void)
 				recvGift(&msg);
 				break;
 			case NET_SCORESUBMIT:				//  a score update from another player
-				recvScoreSubmission(&msg);
+				recvScoreSubmission();
 				break;
 			case NET_VTOL:
 				recvHappyVtol(&msg);

src/multirecv.h

@@ -58,7 +58,7 @@ extern BOOL sendWholeDroid			(DROID	*pD, UDWORD dest);
 //extern BOOL sendWholeStructure		(STRUCTURE *pS, DPID dest);
 extern void recvOptions				(NETMSG *pMsg);
 extern void sendOptions				(UDWORD dest, UDWORD play);
-extern BOOL recvScoreSubmission		(NETMSG *pMsg);
+extern BOOL recvScoreSubmission		();
 extern BOOL recvDestroyExtra		(NETMSG *pMsg);
 extern BOOL	recvAudioMsg			(NETMSG *pMsg);
 extern BOOL recvHappyVtol			(NETMSG *pMsg);

src/multisync.c

@@ -950,62 +950,76 @@ BOOL recvPowerCheck()
 BOOL sendScoreCheck(void)
 {
 	static UDWORD	lastsent = 0;
-	UDWORD			i;
+	uint8_t			i;
+	BOOL			isData = FALSE;
 	PLAYERSTATS		stats;
-	NETMSG			m;
 
-	if(lastsent > gameTime)lastsent= 0;
-	if((gameTime-lastsent) < SCORE_FREQUENCY )	// only send if not done recently.
+	if (lastsent > gameTime)
+	{
+		lastsent= 0;
+	}
+	
+	if (gameTime - lastsent < SCORE_FREQUENCY)
 	{
 		return TRUE;
 	}
+	
 	lastsent = gameTime;
 
-	// syncronise scores.
-
-	// update local score
+	// Update local score
 	stats = getMultiStats(selectedPlayer, TRUE);
 
-	stats.recentKills += stats.killsToAdd;		// add recently scored points.
+	// Add recently scored points
+	stats.recentKills += stats.killsToAdd;
 	stats.totalKills  += stats.killsToAdd;
 	stats.recentScore += stats.scoreToAdd;
 	stats.totalScore  += stats.scoreToAdd;
-	stats.killsToAdd =0;
-	stats.scoreToAdd =0;
 	
-	setMultiStats(player2dpid[selectedPlayer],stats,TRUE);	// store local version.
-	setMultiStats(player2dpid[selectedPlayer],stats,FALSE);	// send score to the ether
+	// Zero them now added
+	stats.killsToAdd = stats.scoreToAdd = 0;
 
-	// broadcast any changes in other players, but not in FRONTEND!!!
+	// Store local version
+	setMultiStats(player2dpid[selectedPlayer], stats, TRUE);
+	
+	// Send score to the ether
+	setMultiStats(player2dpid[selectedPlayer], stats, FALSE);
+
+	// Broadcast any changes in other players, but not in FRONTEND!!!
 	if (titleMode != MULTIOPTION && titleMode != MULTILIMIT)
 	{
-		m.size =0;
+		NETbeginEncode(NET_SCORESUBMIT, NET_ALL_PLAYERS);
+		
 		for (i = 0; i < MAX_PLAYERS; i++)
 		{
-			if(isHumanPlayer(i) && (i!=selectedPlayer))
+			if (isHumanPlayer(i) && i != selectedPlayer)
 			{
 				stats = getMultiStats(i, TRUE);
+				
 				if (stats.killsToAdd || stats.scoreToAdd  )
 				{
-					m.body[m.size] = (UBYTE)i;
-					m.size += 1;
-					NetAdd(m,m.size,stats.killsToAdd);
-					m.size += sizeof(stats.killsToAdd);
-					NetAdd(m,m.size,stats.scoreToAdd);
-					m.size += sizeof(stats.scoreToAdd);
+					NETuint8_t(&i);
+					
+					NETuint32_t(&stats.killsToAdd);
+					NETuint32_t(&stats.scoreToAdd);
+					
+					isData = TRUE;
 				}
 			}
 		}
-		if(m.size != 0)
-		{
-			m.body[m.size] = ANYPLAYER;	// terminate msg.
-			m.size += 1;
-			m.type = NET_SCORESUBMIT;
-			NETbcast(&m,FALSE);
-		}
-	}
 		
-	// get global versions of scores.
+		// If we added any data to the packet
+		if (isData)
+		{
+			// Terminate the message with ANYPLAYER
+			uint8_t player = ANYPLAYER;
+			NETuint8_t(&player);
+			
+			// Send the message
+			NETend();
+		}
+	}
+
+	// Get global versions of scores
 	for (i = 0; i < MAX_PLAYERS; i++)
 	{
 		if (isHumanPlayer(i))
@@ -1013,41 +1027,35 @@ BOOL sendScoreCheck(void)
 			setMultiStats(player2dpid[i], getMultiStats(i, FALSE), TRUE);
 		}
 	}
+	
 	return TRUE;
 }
 
 
 
-BOOL recvScoreSubmission(NETMSG *pMsg)
+BOOL recvScoreSubmission()
 {
-	UDWORD		player=0,kil,index;
-	SDWORD		sco;
+	uint8_t		player;
+	uint32_t	kills, score;
 	PLAYERSTATS	stats;
 
-	// process msg, add to next score addition.
-	index = 0;
-	while(player != ANYPLAYER)
-	{
-		player = pMsg->body[index];
-		index+=1;
-		if(player != ANYPLAYER)
-		{
-			NetGet(pMsg,index,kil);
-			index+=sizeof(kil);
-			NetGet(pMsg,index,sco);
-			index+=sizeof(sco);
+	NETbeginDecode();
 	
-			// do the update.
+	for (NETuint8_t(&player); player != ANYPLAYER; NETuint8_t(&player))
+	{
 		if (player == selectedPlayer)
 		{
+			NETuint32_t(&kills);
+			NETuint32_t(&score);
+			
 			stats = getMultiStats(player, TRUE);
-				stats.killsToAdd += kil;
-				stats.scoreToAdd += sco;
-				setMultiStats(player2dpid[player],stats,TRUE);	// store local version.
+			stats.killsToAdd += kills;
+			stats.scoreToAdd += score;
+			
+			break;
+		}
 	}
 	
-		}
-	}
 	return TRUE;
 }