From cf9447c15b78d84186afb7dc9bde986d415b1014 Mon Sep 17 00:00:00 2001 From: belugas Date: Sat, 9 Feb 2008 02:49:33 +0000 Subject: [PATCH] (svn r12090) -Codechange : removed a magic number and code style application on enum ExpensesType --- src/economy_type.h | 25 +++++++++++++------------ src/player_base.h | 2 +- src/player_gui.cpp | 2 +- 3 files changed, 15 insertions(+), 14 deletions(-) diff --git a/src/economy_type.h b/src/economy_type.h index 9b12f43363..02b515ce43 100644 --- a/src/economy_type.h +++ b/src/economy_type.h @@ -112,18 +112,19 @@ assert_compile(NUM_PRICES * sizeof(Money) == sizeof(Prices)); enum ExpensesType { EXPENSES_CONSTRUCTION = 0, - EXPENSES_NEW_VEHICLES = 1, - EXPENSES_TRAIN_RUN = 2, - EXPENSES_ROADVEH_RUN = 3, - EXPENSES_AIRCRAFT_RUN = 4, - EXPENSES_SHIP_RUN = 5, - EXPENSES_PROPERTY = 6, - EXPENSES_TRAIN_INC = 7, - EXPENSES_ROADVEH_INC = 8, - EXPENSES_AIRCRAFT_INC = 9, - EXPENSES_SHIP_INC = 10, - EXPENSES_LOAN_INT = 11, - EXPENSES_OTHER = 12, + EXPENSES_NEW_VEHICLES, + EXPENSES_TRAIN_RUN, + EXPENSES_ROADVEH_RUN, + EXPENSES_AIRCRAFT_RUN, + EXPENSES_SHIP_RUN, + EXPENSES_PROPERTY, + EXPENSES_TRAIN_INC, + EXPENSES_ROADVEH_INC, + EXPENSES_AIRCRAFT_INC, + EXPENSES_SHIP_INC, + EXPENSES_LOAN_INT, + EXPENSES_OTHER, + EXPENSES_END, INVALID_EXPENSES = 0xFF, }; diff --git a/src/player_base.h b/src/player_base.h index 7bc8540296..43e6460eb9 100644 --- a/src/player_base.h +++ b/src/player_base.h @@ -62,7 +62,7 @@ struct Player { bool is_active; bool is_ai; - Money yearly_expenses[3][13]; + Money yearly_expenses[3][EXPENSES_END]; PlayerEconomyEntry cur_economy; PlayerEconomyEntry old_economy[24]; EngineRenewList engine_renew_list; ///< Defined later diff --git a/src/player_gui.cpp b/src/player_gui.cpp index 01fd4e2332..0bf6815e87 100644 --- a/src/player_gui.cpp +++ b/src/player_gui.cpp @@ -46,7 +46,7 @@ static void DoSelectPlayerFace(PlayerID player, bool show_big); static void DrawPlayerEconomyStats(const Player *p, byte mode) { int x, y, i, j, year; - const Money (*tbl)[13]; + const Money (*tbl)[EXPENSES_END]; Money sum, cost; StringID str;