Make shared memory player state pointers const

Author: slipher

src/engine/qcommon/msg.cpp

@@ -1336,10 +1336,10 @@ MSG_WriteDeltaPlayerstate
 
 =============
 */
-void MSG_WriteDeltaPlayerstate( msg_t *msg, OpaquePlayerState *from, OpaquePlayerState *to )
+void MSG_WriteDeltaPlayerstate(
+	msg_t *msg, const OpaquePlayerState *from, const OpaquePlayerState *to )
 {
 	int           lc;
-	int        *fromF, *toF;
 	float      fullFloat;
 	int        trunc;
 	int        startBit, endBit;
@@ -1383,8 +1383,8 @@ void MSG_WriteDeltaPlayerstate( msg_t *msg, OpaquePlayerState *from, OpaquePlaye
 	for ( int i = 0; i < numFields; i++ )
 	{
 		netField_t* field = &playerStateFields[i];
-		fromF = ( int * )( ( byte * ) from + field->offset );
-		toF = ( int * )( ( byte * ) to + field->offset );
+		auto fromF = reinterpret_cast<const int *>( reinterpret_cast<const byte *>( from ) + field->offset );
+		auto toF = reinterpret_cast<const int *>( reinterpret_cast<const byte *>( to ) + field->offset );
 
 		if (field->bits == STATS_GROUP_FIELD
 			? memcmp(fromF, toF, sizeof(int) * STATS_GROUP_NUM_STATS)
@@ -1401,8 +1401,8 @@ void MSG_WriteDeltaPlayerstate( msg_t *msg, OpaquePlayerState *from, OpaquePlaye
 	for ( int i = 0; i < lc; i++ )
 	{
 		netField_t* field = &playerStateFields[i];
-		fromF = ( int * )( ( byte * ) from + field->offset );
-		toF = ( int * )( ( byte * ) to + field->offset );
+		auto fromF = reinterpret_cast<const int *>( reinterpret_cast<const byte *>( from ) + field->offset );
+		auto toF = reinterpret_cast<const int *>( reinterpret_cast<const byte *>( to ) + field->offset );
 
 		if (field->bits == STATS_GROUP_FIELD)
 		{
@@ -1486,12 +1486,11 @@ static void ReadStatsGroup(msg_t* msg, int* to, const netField_t& field)
 MSG_ReadDeltaPlayerstate
 ===================
 */
-void MSG_ReadDeltaPlayerstate( msg_t *msg, OpaquePlayerState *from, OpaquePlayerState *to )
+void MSG_ReadDeltaPlayerstate( msg_t *msg, const OpaquePlayerState *from, OpaquePlayerState *to )
 {
 	int           lc;
 	int           startBit, endBit;
 	int           print;
-	int           *fromF, *toF;
 	int           trunc;
 
 	if (playerStateFields.empty())
@@ -1537,8 +1536,8 @@ void MSG_ReadDeltaPlayerstate( msg_t *msg, OpaquePlayerState *from, OpaquePlayer
 	for ( int i = 0; i < lc; i++ )
 	{
 		netField_t* field = &playerStateFields[i];
-		fromF = ( int * )( ( byte * ) from + field->offset );
-		toF = ( int * )( ( byte * ) to + field->offset );
+		auto fromF = reinterpret_cast<const int *>( reinterpret_cast<const byte *>( from ) + field->offset );
+		auto toF = reinterpret_cast<int *>( reinterpret_cast<byte *>( to ) + field->offset );
 
 		if ( !MSG_ReadBits( msg, 1 ) )
 		{

src/engine/qcommon/qcommon.h

@@ -96,8 +96,8 @@ void  MSG_WriteDeltaEntity( msg_t *msg, entityState_t *from, entityState_t *to,
 void  MSG_ReadDeltaEntity( msg_t *msg, const entityState_t *from, entityState_t *to, int number );
 
 void MSG_InitNetcodeTables(NetcodeTable playerStateTable, int playerStateSize);
-void  MSG_WriteDeltaPlayerstate( msg_t *msg, OpaquePlayerState *from, OpaquePlayerState *to );
-void  MSG_ReadDeltaPlayerstate( msg_t *msg, OpaquePlayerState *from, OpaquePlayerState *to );
+void  MSG_WriteDeltaPlayerstate( msg_t *msg, const OpaquePlayerState *from, const OpaquePlayerState *to );
+void  MSG_ReadDeltaPlayerstate( msg_t *msg, const OpaquePlayerState *from, OpaquePlayerState *to );
 
 //============================================================================
 

src/engine/server/server.h

@@ -82,12 +82,12 @@ struct server_t
 	const char            *entityParsePoint; // used during game VM init
 
 	// the game virtual machine will update these on init and changes
-	sharedEntity_t *gentities;
-	int            gentitySize;
+	byte *gentities; // points to the sgame's array of gentity_t
+	int            gentitySize; // >= sizeof(sharedEntity_t) - game can have unlimited amount of private data
 	int            num_entities; // current number, <= MAX_GENTITIES
 
-	OpaquePlayerState *gameClients;
-	int            gameClientSize; // will be > sizeof(playerState_t) due to game private data
+	const byte *gameClients; // points to the sgame's array of playerState_t in shared memory
+	int            gameClientSize; // will be <= sizeof(OpaquePlayerState)
 
 	int            restartTime;
 	int            time;
@@ -393,7 +393,7 @@ void SV_SendClientIdle( client_t *client );
 // sv_sgame.c
 //
 sharedEntity_t *SV_GentityNum( int num );
-OpaquePlayerState *SV_GameClientNum( int num );
+const OpaquePlayerState *SV_GameClientNum( int num );
 
 svEntity_t     *SV_SvEntityForGentity( sharedEntity_t *gEnt );
 void           SV_InitGameProgs();

src/engine/server/sv_ccmds.cpp

@@ -338,7 +338,7 @@ class StatusCmd: public Cmd::StaticCmd
 					connection = "ERROR";
 				}
 			}
-			OpaquePlayerState* ps = SV_GameClientNum( i );
+			const OpaquePlayerState* ps = SV_GameClientNum( i );
 
 			const char *address = NET_AdrToString( cl.netchan.remoteAddress );
 

src/engine/server/sv_main.cpp

@@ -496,7 +496,7 @@ static void SVC_Status( const netadr_t& from, const Cmd::Args& args )
 
 		if ( cl->state >= clientState_t::CS_CONNECTED )
 		{
-			OpaquePlayerState* ps = SV_GameClientNum( i );
+			const OpaquePlayerState* ps = SV_GameClientNum( i );
 			status +=  Str::Format( "%i %i \"%s\"\n", ps->persistant[ PERS_SCORE ], cl->ping, cl->name );
 		}
 	}

src/engine/server/sv_sgame.cpp

@@ -60,10 +60,10 @@ sharedEntity_t *SV_GentityNum( int num )
 		Sys::Drop( "SV_GentityNum: bad num %d", num );
 	}
 
-	return ( sharedEntity_t * )( ( byte * ) sv.gentities + sv.gentitySize * ( num ) );
+	return reinterpret_cast<sharedEntity_t *>( sv.gentities + sv.gentitySize * num );
 }
 
-OpaquePlayerState *SV_GameClientNum( int num )
+const OpaquePlayerState *SV_GameClientNum( int num )
 {
 	if ( num < 0 || num >= sv_maxclients->integer || sv.gameClients == nullptr )
 	{
@@ -194,12 +194,12 @@ void SV_LocateGameData( const IPC::SharedMemory& shmRegion, int numGEntities, in
 	if ( int64_t(shmRegion.GetSize()) < int64_t(MAX_GENTITIES) * sizeofGEntity_t + int64_t(sv_maxclients->integer) * sizeofGameClient )
 		Sys::Drop( "SV_LocateGameData: Shared memory region too small" );
 
-	char* base = static_cast<char*>(shmRegion.GetBase());
-	sv.gentities = reinterpret_cast<sharedEntity_t*>(base);
+	byte* base = static_cast<byte*>(shmRegion.GetBase());
+	sv.gentities = base;
 	sv.gentitySize = sizeofGEntity_t;
 	sv.num_entities = numGEntities;
 
-	sv.gameClients = reinterpret_cast<OpaquePlayerState*>(base + MAX_GENTITIES * size_t(sizeofGEntity_t));
+	sv.gameClients = base + MAX_GENTITIES * size_t(sizeofGEntity_t);
 	sv.gameClientSize = sizeofGameClient;
 }
 

src/engine/server/sv_snapshot.cpp

@@ -690,7 +690,7 @@ static void SV_BuildClientSnapshot( client_t *client )
 	}
 
 	// grab the current playerState_t
-	OpaquePlayerState* ps = SV_GameClientNum( client - svs.clients );
+	const OpaquePlayerState* ps = SV_GameClientNum( client - svs.clients );
 	memcpy(&frame->ps, ps, sizeof(frame->ps));
 
 	// never send client's own entity, because it can