diff options
author | Darkvater <Darkvater@openttd.org> | 2006-10-12 11:42:57 +0000 |
---|---|---|
committer | Darkvater <Darkvater@openttd.org> | 2006-10-12 11:42:57 +0000 |
commit | 40365cc2924a2a0bcff6732a44432845b6fe6cf1 (patch) | |
tree | 36f9f8314abfadad8df25176c1475acfe3b83c12 | |
parent | afdc0c2e575e46161bed3267d8112dc9c98a66c7 (diff) | |
download | openttd-40365cc2924a2a0bcff6732a44432845b6fe6cf1.tar.xz |
(svn r6748) -Codechange: Use already existing function to count active players.
-rw-r--r-- | main_gui.c | 7 | ||||
-rw-r--r-- | player.h | 12 | ||||
-rw-r--r-- | players.c | 12 |
3 files changed, 12 insertions, 19 deletions
diff --git a/main_gui.c b/main_gui.c index 3254ffeb5..9726239c3 100644 --- a/main_gui.c +++ b/main_gui.c @@ -531,12 +531,7 @@ static int GetPlayerIndexFromMenu(int index) static void UpdatePlayerMenuHeight(Window *w) { - uint num = 0; - const Player *p; - - FOR_ALL_PLAYERS(p) { - if (p->is_active) num++; - } + byte num = ActivePlayerCount(); // Increase one to fit in PlayerList in the menu when in network if (_networking && WP(w,menu_d).main_button == 9) num++; @@ -216,7 +216,17 @@ VARDEF Player _players[MAX_PLAYERS]; // NOSAVE: can be determined from player structs VARDEF byte _player_colors[MAX_PLAYERS]; -byte ActivePlayerCount(void); +static inline byte ActivePlayerCount(void) +{ + const Player *p; + byte count = 0; + + FOR_ALL_PLAYERS(p) { + if (p->is_active) count++; + } + + return count; +} static inline Player* GetPlayer(PlayerID i) { @@ -192,18 +192,6 @@ void DrawPlayerFace(uint32 face, int color, int x, int y) } } -byte ActivePlayerCount(void) -{ - const Player *p; - byte count = 0; - - FOR_ALL_PLAYERS(p) { - if (p->is_active) count++; - } - - return count; -} - void InvalidatePlayerWindows(const Player *p) { PlayerID pid = p->index; |