diff options
| author | Darkvater <darkvater@openttd.org> | 2006-10-14 22:31:18 +0000 |
|---|---|---|
| committer | Darkvater <darkvater@openttd.org> | 2006-10-14 22:31:18 +0000 |
| commit | 6b6d908308c9aaf32f7b99500715319aaab1a51c (patch) | |
| tree | 52f21281bd10b45ef340a15357a04bf130b9b554 /misc_gui.c | |
| parent | 9732c129c16dea48e69188b75c2e39f9faacf86f (diff) | |
| download | openttd-6b6d908308c9aaf32f7b99500715319aaab1a51c.tar.xz | |
(svn r6776) -Codechange: Use IsValidPlayer() function to determine of a PlayerID is an
actual playable player (< MAX_PLAYERS) or not.
Diffstat (limited to 'misc_gui.c')
| -rw-r--r-- | misc_gui.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/misc_gui.c b/misc_gui.c index a812d5210..aece882c5 100644 --- a/misc_gui.c +++ b/misc_gui.c @@ -155,7 +155,7 @@ static void Place_LandInfo(TileIndex tile) lid.tile = tile; lid.town = ClosestTownFromTile(tile, _patches.dist_local_authority); - p = GetPlayer(_local_player < MAX_PLAYERS ? _local_player : 0); + p = GetPlayer(IsValidPlayer(_local_player) ? _local_player : 0); old_money = p->money64; p->money64 = p->player_money = 0x7fffffff; @@ -1306,7 +1306,7 @@ static void GenerateFileName(void) { /* Check if we are not a specatator who wants to generate a name.. Let's use the name of player #0 for now. */ - const Player *p = GetPlayer(_local_player < MAX_PLAYERS ? _local_player : 0); + const Player *p = GetPlayer(IsValidPlayer(_local_player) ? _local_player : 0); SetDParam(0, p->name_1); SetDParam(1, p->name_2); @@ -1636,9 +1636,9 @@ static int32 ClickMoneyCheat(int32 p1, int32 p2) // p1 player to set to, p2 is -1 or +1 (down/up) static int32 ClickChangePlayerCheat(int32 p1, int32 p2) { - while (p1 >= 0 && p1 < MAX_PLAYERS) { + while (IsValidPlayer((PlayerID)p1)) { if (_players[p1].is_active) { - _local_player = p1; + _local_player = (PlayerID)p1; MarkWholeScreenDirty(); return _local_player; } |