(svn r8272) -Fix (r8038): signed/unsigned change - PlayerID is now enum (signed) and was byte so test for PlayerID validity needed to be updated (Darkvater)

1 files changed, 2 insertions, 2 deletions

diff --git a/src/player.h b/src/player.h
index e3329c0d1..1f91f58de 100644
--- a/src/player.h
+++ b/src/player.h
@@ -230,7 +230,7 @@ static inline byte ActivePlayerCount(void)
 
 static inline Player* GetPlayer(PlayerID i)
 {
-	assert(i < (PlayerID)lengthof(_players));
+	assert(IS_INSIDE_1D(i, PLAYER_FIRST, lengthof(_players)));
 	return &_players[i];
 }
 
@@ -241,7 +241,7 @@ static inline bool IsLocalPlayer(void)
 
 static inline bool IsValidPlayer(PlayerID pi)
 {
-	return pi < MAX_PLAYERS;
+	return IS_INSIDE_1D(pi, PLAYER_FIRST, MAX_PLAYERS);
 }
 
 byte GetPlayerRailtypes(PlayerID p);