summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--aircraft_gui.c12
-rw-r--r--economy.c6
-rw-r--r--industry_cmd.c6
-rw-r--r--landscape.c2
-rw-r--r--misc_gui.c8
-rw-r--r--network_data.h2
-rw-r--r--news_gui.c2
-rw-r--r--oldloader.c2
-rw-r--r--openttd.c2
-rw-r--r--player_gui.c2
-rw-r--r--players.c4
-rw-r--r--roadveh_gui.c12
-rw-r--r--ship_gui.c12
-rw-r--r--signs.h2
-rw-r--r--station.h4
-rw-r--r--station_cmd.c34
-rw-r--r--station_gui.c28
-rw-r--r--town.h2
-rw-r--r--town_cmd.c4
-rw-r--r--train_gui.c15
-rw-r--r--vehicle.c4
-rw-r--r--window.h4
22 files changed, 84 insertions, 85 deletions
diff --git a/aircraft_gui.c b/aircraft_gui.c
index b01b52129..13814a4b8 100644
--- a/aircraft_gui.c
+++ b/aircraft_gui.c
@@ -129,7 +129,7 @@ static void NewAircraftWndProc(Window *w, WindowEvent *e)
int sel = WP(w,buildtrain_d).sel_index;
int pos = w->vscroll.pos;
EngineID engine_id = AIRCRAFT_ENGINES_INDEX;
- int selected_id = -1;
+ EngineID selected_id = INVALID_ENGINE;
do {
if (HASBIT(e->player_avail, _local_player)) {
@@ -145,7 +145,7 @@ static void NewAircraftWndProc(Window *w, WindowEvent *e)
WP(w,buildtrain_d).sel_engine = selected_id;
- if (selected_id != -1) {
+ if (selected_id != INVALID_ENGINE) {
DrawAircraftPurchaseInfo(2, w->widget[4].top + 1, selected_id);
}
}
@@ -162,14 +162,14 @@ static void NewAircraftWndProc(Window *w, WindowEvent *e)
} break;
case 5: { /* build */
- int sel_eng = WP(w,buildtrain_d).sel_engine;
- if (sel_eng != -1)
+ EngineID sel_eng = WP(w,buildtrain_d).sel_engine;
+ if (sel_eng != INVALID_ENGINE)
DoCommandP(w->window_number, sel_eng, 0, CcBuildAircraft, CMD_BUILD_AIRCRAFT | CMD_MSG(STR_A008_CAN_T_BUILD_AIRCRAFT));
} break;
case 6: { /* rename */
- int sel_eng = WP(w,buildtrain_d).sel_engine;
- if (sel_eng != -1) {
+ EngineID sel_eng = WP(w,buildtrain_d).sel_engine;
+ if (sel_eng != INVALID_ENGINE) {
WP(w,buildtrain_d).rename_engine = sel_eng;
ShowQueryString(GetCustomEngineName(sel_eng),
STR_A039_RENAME_AIRCRAFT_TYPE, 31, 160, w->window_class, w->window_number);
diff --git a/economy.c b/economy.c
index 10676e417..7acd051a0 100644
--- a/economy.c
+++ b/economy.c
@@ -1323,7 +1323,7 @@ int LoadUnloadVehicle(Vehicle *v)
GoodsEntry *ge;
int t;
uint count, cap;
- byte old_player;
+ PlayerID old_player;
bool completely_empty = true;
assert(v->current_order.type == OT_LOADING);
@@ -1560,8 +1560,8 @@ int32 CmdBuyShareInCompany(int x, int y, uint32 flags, uint32 p1, uint32 p2)
cost = CalculateCompanyValue(p) >> 2;
if (flags & DC_EXEC) {
+ PlayerID* b = p->share_owners;
int i;
- byte *b = p->share_owners;
while (*b != OWNER_SPECTATOR) b++; /* share owners is guaranteed to contain at least one OWNER_SPECTATOR */
*b = _current_player;
@@ -1602,7 +1602,7 @@ int32 CmdSellShareInCompany(int x, int y, uint32 flags, uint32 p1, uint32 p2)
cost = -(cost - (cost >> 7));
if (flags & DC_EXEC) {
- byte *b = p->share_owners;
+ PlayerID* b = p->share_owners;
while (*b != _current_player) b++; /* share owners is guaranteed to contain player */
*b = OWNER_SPECTATOR;
InvalidateWindow(WC_COMPANY, (int)p1);
diff --git a/industry_cmd.c b/industry_cmd.c
index 7ec6c82aa..953c0fa4f 100644
--- a/industry_cmd.c
+++ b/industry_cmd.c
@@ -1042,7 +1042,7 @@ static void ChopLumberMillTrees(Industry *i)
do {
tile = TILE_MASK(tile);
if (IsTileType(tile, MP_TREES)) {
- uint old_player = _current_player;
+ PlayerID old_player = _current_player;
/* found a tree */
_current_player = OWNER_NONE;
@@ -1662,7 +1662,7 @@ static void PlaceInitialIndustry(byte type, int amount)
if (_opt.diff.number_industries != 0)
{
- byte old_player = _current_player;
+ PlayerID old_player = _current_player;
_current_player = OWNER_NONE;
assert(num > 0);
@@ -1902,7 +1902,7 @@ static void ChangeIndustryProduction(Industry *i)
void IndustryMonthlyLoop(void)
{
Industry *i;
- byte old_player = _current_player;
+ PlayerID old_player = _current_player;
_current_player = OWNER_NONE;
FOR_ALL_INDUSTRIES(i) {
diff --git a/landscape.c b/landscape.c
index 31bfe68cb..c884b0b3d 100644
--- a/landscape.c
+++ b/landscape.c
@@ -380,7 +380,7 @@ void CDECL ModifyTile(TileIndex tile, uint flags, ...)
}
if (flags & (MP_MAPOWNER|MP_MAPOWNER_CURRENT)) {
- byte x = _current_player;
+ PlayerID x = _current_player;
if (flags & MP_MAPOWNER) x = va_arg(va, int);
_m[tile].m1 = x;
}
diff --git a/misc_gui.c b/misc_gui.c
index 00062b5e2..2701ec4e9 100644
--- a/misc_gui.c
+++ b/misc_gui.c
@@ -583,19 +583,19 @@ void ShowErrorMessage(StringID msg_1, StringID msg_2, int x, int y)
void ShowEstimatedCostOrIncome(int32 cost, int x, int y)
{
- int msg = STR_0805_ESTIMATED_COST;
+ StringID msg = STR_0805_ESTIMATED_COST;
if (cost < 0) {
cost = -cost;
msg = STR_0807_ESTIMATED_INCOME;
}
SetDParam(0, cost);
- ShowErrorMessage(-1, msg, x, y);
+ ShowErrorMessage(INVALID_STRING_ID, msg, x, y);
}
void ShowCostOrIncomeAnimation(int x, int y, int z, int32 cost)
{
- int msg;
+ StringID msg;
Point pt = RemapCoords(x,y,z);
msg = STR_0801_COST;
@@ -712,7 +712,7 @@ static void DrawStationCoverageText(const AcceptedCargo accepts,
void DrawStationCoverageAreaText(int sx, int sy, uint mask, int rad) {
int x = _thd.pos.x;
int y = _thd.pos.y;
- uint accepts[NUM_CARGO];
+ AcceptedCargo accepts;
if (x != -1) {
GetAcceptanceAroundTiles(accepts, TileVirtXY(x, y), _thd.size.x / 16, _thd.size.y / 16 , rad);
DrawStationCoverageText(accepts, sx, sy, mask);
diff --git a/network_data.h b/network_data.h
index a8d748c42..b4dea28b1 100644
--- a/network_data.h
+++ b/network_data.h
@@ -35,7 +35,7 @@ typedef struct Packet {
typedef struct CommandPacket {
struct CommandPacket *next;
- byte player; /// player that is executing the command (PlayerID)
+ PlayerID player; /// player that is executing the command
uint32 cmd; /// command being executed
uint32 p1; /// parameter p1
uint32 p2; /// parameter p2
diff --git a/news_gui.c b/news_gui.c
index 612483174..bd26e7310 100644
--- a/news_gui.c
+++ b/news_gui.c
@@ -373,7 +373,7 @@ static inline void SetNewsDisplayValue(byte item, byte val)
static void ShowNewspaper(NewsItem *ni)
{
Window *w;
- int sound;
+ SoundFx sound;
int top;
ni->flags &= ~(NF_NOEXPIRE | NF_FORCE_BIG);
ni->duration = 555;
diff --git a/oldloader.c b/oldloader.c
index 20f409bda..214d0dddf 100644
--- a/oldloader.c
+++ b/oldloader.c
@@ -789,7 +789,7 @@ static bool LoadOldIndustry(LoadgameState *ls, int num)
return true;
}
-static uint _current_player_id;
+static PlayerID _current_player_id;
static uint16 _old_inaugurated_year;
static int32 _old_yearly;
diff --git a/openttd.c b/openttd.c
index 5b33dd1f2..0e06fed7e 100644
--- a/openttd.c
+++ b/openttd.c
@@ -867,7 +867,7 @@ void StateGameLoop(void)
} else {
// All these actions has to be done from OWNER_NONE
// for multiplayer compatibility
- uint p = _current_player;
+ PlayerID p = _current_player;
_current_player = OWNER_NONE;
AnimateAnimatedTiles();
diff --git a/player_gui.c b/player_gui.c
index 41fdf9e92..39f7642d0 100644
--- a/player_gui.c
+++ b/player_gui.c
@@ -691,7 +691,7 @@ static const WindowDesc _other_player_company_desc = {
void ShowPlayerCompany(PlayerID player)
{
Window *w;
- w = AllocateWindowDescFront((byte)player == _local_player ? &_my_player_company_desc : &_other_player_company_desc, player);
+ w = AllocateWindowDescFront(player == _local_player ? &_my_player_company_desc : &_other_player_company_desc, player);
if (w)
w->caption_color = w->window_number;
}
diff --git a/players.c b/players.c
index 4ae340a19..7169713cf 100644
--- a/players.c
+++ b/players.c
@@ -196,7 +196,7 @@ void InvalidatePlayerWindows(const Player *p)
bool CheckPlayerHasMoney(int32 cost)
{
if (cost > 0) {
- uint pid = _current_player;
+ PlayerID pid = _current_player;
if (pid < MAX_PLAYERS && cost > GetPlayer(pid)->player_money) {
SetDParam(0, cost);
_error_message = STR_0003_NOT_ENOUGH_CASH_REQUIRES;
@@ -284,7 +284,7 @@ bool CheckOwnership(PlayerID owner)
bool CheckTileOwnership(TileIndex tile)
{
- byte owner = GetTileOwner(tile);
+ PlayerID owner = GetTileOwner(tile);
assert(owner <= OWNER_WATER);
diff --git a/roadveh_gui.c b/roadveh_gui.c
index 8d82a2949..b812594e1 100644
--- a/roadveh_gui.c
+++ b/roadveh_gui.c
@@ -412,7 +412,7 @@ static void DrawNewRoadVehWindow(Window *w)
int sel = WP(w,buildtrain_d).sel_index;
int pos = w->vscroll.pos;
EngineID engine_id = ROAD_ENGINES_INDEX;
- int selected_id = -1;
+ EngineID selected_id = INVALID_ENGINE;
do {
if (HASBIT(e->player_avail, _local_player)) {
@@ -427,7 +427,7 @@ static void DrawNewRoadVehWindow(Window *w)
} while (++engine_id, ++e,--num);
WP(w,buildtrain_d).sel_engine = selected_id;
- if (selected_id != -1) {
+ if (selected_id != INVALID_ENGINE) {
DrawRoadVehPurchaseInfo(2, w->widget[4].top + 1, selected_id);
}
}
@@ -465,14 +465,14 @@ static void NewRoadVehWndProc(Window *w, WindowEvent *e)
} break;
case 5: { /* build */
- int sel_eng = WP(w,buildtrain_d).sel_engine;
- if (sel_eng != -1)
+ EngineID sel_eng = WP(w,buildtrain_d).sel_engine;
+ if (sel_eng != INVALID_ENGINE)
DoCommandP(w->window_number, sel_eng, 0, CcBuildRoadVeh, CMD_BUILD_ROAD_VEH | CMD_MSG(STR_9009_CAN_T_BUILD_ROAD_VEHICLE));
} break;
case 6: { /* rename */
- int sel_eng = WP(w,buildtrain_d).sel_engine;
- if (sel_eng != -1) {
+ EngineID sel_eng = WP(w,buildtrain_d).sel_engine;
+ if (sel_eng != INVALID_ENGINE) {
WP(w,buildtrain_d).rename_engine = sel_eng;
ShowQueryString(GetCustomEngineName(sel_eng),
STR_9036_RENAME_ROAD_VEHICLE_TYPE, 31, 160, w->window_class, w->window_number);
diff --git a/ship_gui.c b/ship_gui.c
index 898591482..c9ff4c5df 100644
--- a/ship_gui.c
+++ b/ship_gui.c
@@ -352,7 +352,7 @@ static void NewShipWndProc(Window *w, WindowEvent *e)
int sel = WP(w,buildtrain_d).sel_index;
int pos = w->vscroll.pos;
EngineID engine_id = SHIP_ENGINES_INDEX;
- int selected_id = -1;
+ EngineID selected_id = INVALID_ENGINE;
do {
if (HASBIT(e->player_avail, _local_player)) {
@@ -368,7 +368,7 @@ static void NewShipWndProc(Window *w, WindowEvent *e)
WP(w,buildtrain_d).sel_engine = selected_id;
- if (selected_id != -1) {
+ if (selected_id != INVALID_ENGINE) {
DrawShipPurchaseInfo(2, w->widget[4].top + 1, selected_id);
}
}
@@ -384,14 +384,14 @@ static void NewShipWndProc(Window *w, WindowEvent *e)
}
} break;
case 5: { /* build */
- int sel_eng = WP(w,buildtrain_d).sel_engine;
- if (sel_eng != -1)
+ EngineID sel_eng = WP(w,buildtrain_d).sel_engine;
+ if (sel_eng != INVALID_ENGINE)
DoCommandP(w->window_number, sel_eng, 0, CcBuildShip, CMD_BUILD_SHIP | CMD_MSG(STR_980D_CAN_T_BUILD_SHIP));
} break;
case 6: { /* rename */
- int sel_eng = WP(w,buildtrain_d).sel_engine;
- if (sel_eng != -1) {
+ EngineID sel_eng = WP(w,buildtrain_d).sel_engine;
+ if (sel_eng != INVALID_ENGINE) {
WP(w,buildtrain_d).rename_engine = sel_eng;
ShowQueryString(GetCustomEngineName(sel_eng),
STR_9838_RENAME_SHIP_TYPE, 31, 160, w->window_class, w->window_number);
diff --git a/signs.h b/signs.h
index cdb00c49a..7c6b92ec0 100644
--- a/signs.h
+++ b/signs.h
@@ -11,7 +11,7 @@ typedef struct SignStruct {
int32 x;
int32 y;
byte z;
- byte owner; // placed by this player. Anyone can delete them though.
+ PlayerID owner; // placed by this player. Anyone can delete them though.
// OWNER_NONE for gray signs from old games.
uint16 index;
diff --git a/station.h b/station.h
index 26f235225..f7d7850ac 100644
--- a/station.h
+++ b/station.h
@@ -123,11 +123,11 @@ enum {
CA_AIR_INTER = 8,
};
-void ModifyStationRatingAround(TileIndex tile, byte owner, int amount, uint radius);
+void ModifyStationRatingAround(TileIndex tile, PlayerID owner, int amount, uint radius);
TileIndex GetStationTileForVehicle(const Vehicle *v, const Station *st);
-void ShowStationViewWindow(int station);
+void ShowStationViewWindow(StationID station);
void UpdateAllStationVirtCoord(void);
VARDEF SortStruct *_station_sort;
diff --git a/station_cmd.c b/station_cmd.c
index c012f550c..e5e51d459 100644
--- a/station_cmd.c
+++ b/station_cmd.c
@@ -179,13 +179,12 @@ static byte FindCatchmentRadius(Station *st)
#define CHECK_STATIONS_ERR ((Station*)-1)
-static Station *GetStationAround(TileIndex tile, int w, int h, int closest_station)
+static Station* GetStationAround(TileIndex tile, int w, int h, StationID closest_station)
{
// check around to see if there's any stations there
BEGIN_TILE_LOOP(tile_cur, w + 2, h + 2, tile - TileDiffXY(1, 1))
if (IsTileType(tile_cur, MP_STATION)) {
- int t;
- t = _m[tile_cur].m2;
+ StationID t = _m[tile_cur].m2;
{
Station *st = GetStation(t);
// you cannot take control of an oilrig!!
@@ -193,7 +192,7 @@ static Station *GetStationAround(TileIndex tile, int w, int h, int closest_stati
continue;
}
- if (closest_station == -1) {
+ if (closest_station == INVALID_STATION) {
closest_station = t;
} else if (closest_station != t) {
_error_message = STR_3006_ADJOINS_MORE_THAN_ONE_EXISTING;
@@ -201,7 +200,7 @@ static Station *GetStationAround(TileIndex tile, int w, int h, int closest_stati
}
}
END_TILE_LOOP(tile_cur, w + 2, h + 2, tile - TileDiffXY(1, 1))
- return (closest_station == -1) ? NULL : GetStation(closest_station);
+ return (closest_station == INVALID_STATION) ? NULL : GetStation(closest_station);
}
TileIndex GetStationTileForVehicle(const Vehicle *v, const Station *st)
@@ -424,7 +423,7 @@ done:
}
#undef M
-static Station *GetClosestStationFromTile(TileIndex tile, uint threshold, byte owner)
+static Station* GetClosestStationFromTile(TileIndex tile, uint threshold, PlayerID owner)
{
Station* best_station = NULL;
Station* st;
@@ -755,7 +754,7 @@ static int32 ClearTile_Station(TileIndex tile, byte flags);
// Tries to clear the given area. Returns the cost in case of success.
// Or an error code if it failed.
-int32 CheckFlatLandBelow(TileIndex tile, uint w, uint h, uint flags, uint invalid_dirs, int *station)
+int32 CheckFlatLandBelow(TileIndex tile, uint w, uint h, uint flags, uint invalid_dirs, StationID* station)
{
int32 cost = 0, ret;
@@ -810,17 +809,17 @@ int32 CheckFlatLandBelow(TileIndex tile, uint w, uint h, uint flags, uint invali
}
// if station is set, then we have special handling to allow building on top of already existing stations.
- // so station points to -1 if we can build on any station. or it points to a station if we're only allowed to build
+ // so station points to INVALID_STATION if we can build on any station. or it points to a station if we're only allowed to build
// on exactly that station.
if (station != NULL && IsTileType(tile_cur, MP_STATION)) {
if (_m[tile_cur].m5 >= 8) {
_error_message = ClearTile_Station(tile_cur, DC_AUTO); // get error message
return CMD_ERROR;
} else {
- int st = _m[tile_cur].m2;
- if (*station == -1)
+ StationID st = _m[tile_cur].m2;
+ if (*station == INVALID_STATION) {
*station = st;
- else if (*station != st) {
+ } else if (*station != st) {
_error_message = STR_3006_ADJOINS_MORE_THAN_ONE_EXISTING;
return CMD_ERROR;
}
@@ -952,7 +951,7 @@ int32 CmdBuildRailroadStation(int x, int y, uint32 flags, uint32 p1, uint32 p2)
TileIndex tile_org;
int w_org, h_org;
int32 cost, ret;
- int est;
+ StationID est;
int plat_len, numtracks;
int direction;
uint finalvalues[3];
@@ -986,7 +985,7 @@ int32 CmdBuildRailroadStation(int x, int y, uint32 flags, uint32 p1, uint32 p2)
finalvalues[2] = h_org;
// Make sure the area below consists of clear tiles. (OR tiles belonging to a certain rail station)
- est = -1;
+ est = INVALID_STATION;
// If DC_EXEC is in flag, do not want to pass it to CheckFlatLandBelow, because of a nice bug
// for detail info, see: https://sourceforge.net/tracker/index.php?func=detail&aid=1029064&group_id=103924&atid=636365
if (CmdFailed(ret = CheckFlatLandBelow(tile_org, w_org, h_org, flags&~DC_EXEC, 5 << direction, _patches.nonuniform_stations ? &est : NULL))) return CMD_ERROR;
@@ -1190,8 +1189,9 @@ int32 CmdRemoveFromRailroadStation(int x, int y, uint32 flags, uint32 p1, uint32
// determine the number of platforms for the station
uint GetStationPlatforms(const Station *st, TileIndex tile)
{
- uint t;
- int dir,delta;
+ TileIndex t;
+ TileIndexDiff delta;
+ int dir;
int len;
assert(TileBelongsToRailStation(st, tile));
@@ -2137,7 +2137,7 @@ static void DrawTile_Station(TileInfo *ti)
uint32 relocation = 0;
{
- uint owner = GetTileOwner(ti->tile);
+ PlayerID owner = GetTileOwner(ti->tile);
image_or_modificator = PALETTE_TO_GREY; /* NOTE: possible bug in ttd here? */
if (owner < MAX_PLAYERS)
image_or_modificator = PLAYER_SPRITE_COLOR(owner);
@@ -2680,7 +2680,7 @@ void StationMonthlyLoop(void)
}
-void ModifyStationRatingAround(TileIndex tile, byte owner, int amount, uint radius)
+void ModifyStationRatingAround(TileIndex tile, PlayerID owner, int amount, uint radius)
{
Station *st;
GoodsEntry *ge;
diff --git a/station_gui.c b/station_gui.c
index 98912fb08..be1376b32 100644
--- a/station_gui.c
+++ b/station_gui.c
@@ -112,7 +112,7 @@ static void GlobalSortStationList(void)
DEBUG(misc, 1) ("Resorting global station list...");
}
-static void MakeSortedStationList(byte owner)
+static void MakeSortedStationList(PlayerID owner)
{
SortStruct *firstelement;
uint32 n = 0;
@@ -137,25 +137,25 @@ static void PlayerStationsWndProc(Window *w, WindowEvent *e)
{
switch(e->event) {
case WE_PAINT: {
+ const PlayerID owner = w->window_number;
uint32 i;
- const byte window_number = (byte)w->window_number;
// resort station window if stations have been added/removed
if (_global_station_sort_dirty)
GlobalSortStationList();
- if (_station_sort_dirty[window_number]) { // resort in case of a station rename.
- MakeSortedStationList(window_number);
+ if (_station_sort_dirty[owner]) { // resort in case of a station rename.
+ MakeSortedStationList(owner);
}
// stations are stored as a cummulative index, eg 25, 41, 43. This means
// Player0: 25; Player1: (41-25) 16; Player2: (43-41) 2 stations
- i = (window_number == 0) ? 0 : _num_station_sort[window_number-1];
- SetVScrollCount(w, _num_station_sort[window_number] - i);
+ i = (owner == 0) ? 0 : _num_station_sort[owner - 1];
+ SetVScrollCount(w, _num_station_sort[owner] - i);
/* draw widgets, with player's name in the caption */
{
- Player *p = GetPlayer(window_number);
+ const Player* p = GetPlayer(owner);
SetDParam(0, p->name_1);
SetDParam(1, p->name_2);
SetDParam(2, w->vscroll.count);
@@ -175,12 +175,12 @@ static void PlayerStationsWndProc(Window *w, WindowEvent *e)
}
i += w->vscroll.pos; // offset from sorted station list of current player
- assert(i < _num_station_sort[window_number]); // at least one station must exist
+ assert(i < _num_station_sort[owner]); // at least one station must exist
- while (i < _num_station_sort[window_number]) { // do until max number of stations of owner
+ while (i < _num_station_sort[owner]) { // do until max number of stations of owner
st = GetStation(_station_sort[i].index);
- assert(st->xy && st->owner == window_number);
+ assert(st->xy && st->owner == owner);
SetDParam(0, st->index);
SetDParam(1, st->facilities);
@@ -210,7 +210,7 @@ static void PlayerStationsWndProc(Window *w, WindowEvent *e)
id_v += w->vscroll.pos;
{
- const byte owner = (byte)w->window_number;
+ const PlayerID owner = w->window_number;
Station *st;
id_v += (owner == 0) ? 0 : _num_station_sort[owner - 1]; // first element in list
@@ -316,11 +316,11 @@ static void DrawStationViewWindow(Window *w)
int x,y;
int pos;
StringID str;
- uint16 station_id;
+ StationID station_id;
station_id = w->window_number;
- st = GetStation(w->window_number);
+ st = GetStation(station_id);
num = 1;
for(i=0; i!=NUM_CARGO; i++) {
@@ -533,7 +533,7 @@ static const WindowDesc _station_view_desc = {
StationViewWndProc
};
-void ShowStationViewWindow(int station)
+void ShowStationViewWindow(StationID station)
{
Window *w;
byte color;
diff --git a/town.h b/town.h
index 89a5cbb1c..906114725 100644
--- a/town.h
+++ b/town.h
@@ -32,7 +32,7 @@ struct Town {
// Player ratings as well as a mask that determines which players have a rating.
byte have_ratings;
uint8 unwanted[MAX_PLAYERS]; // how many months companies aren't wanted by towns (bribe)
- uint8 exclusivity; // which player has exslusivity
+ PlayerID exclusivity; // which player has exslusivity
uint8 exclusive_counter; // months till the exclusivity expires
int16 ratings[MAX_PLAYERS];
diff --git a/town_cmd.c b/town_cmd.c
index 9749cee8e..6ee747217 100644
--- a/town_cmd.c
+++ b/town_cmd.c
@@ -782,7 +782,7 @@ bool GrowTown(Town *t)
TileIndex tile;
const TileIndexDiffC *ptr;
TileInfo ti;
- byte old_player;
+ PlayerID old_player;
static const TileIndexDiffC _town_coord_mod[] = {
{-1, 0},
@@ -1578,7 +1578,7 @@ static void TownActionRoadRebuild(Town *t, int action)
static bool DoBuildStatueOfCompany(TileIndex tile)
{
TileInfo ti;
- byte old;
+ PlayerID old;
int32 r;
FindLandscapeHeightByTile(&ti, tile);
diff --git a/train_gui.c b/train_gui.c
index fe64fcccc..67968945a 100644
--- a/train_gui.c
+++ b/train_gui.c
@@ -173,7 +173,7 @@ void CcCloneTrain(bool success, uint tile, uint32 p1, uint32 p2)
}
static void engine_drawing_loop(int *x, int *y, int *pos, int *sel,
- int *selected_id, byte railtype, byte show_max, bool is_engine)
+ EngineID* selected_id, byte railtype, byte show_max, bool is_engine)
{
EngineID i;
@@ -229,7 +229,7 @@ static void NewRailVehicleWndProc(Window *w, WindowEvent *e)
int pos = w->vscroll.pos;
int x = 1;
int y = 15;
- int selected_id = -1;
+ EngineID selected_id = INVALID_ENGINE;
/* Ensure that custom engines which substituted wagons
* are sorted correctly.
@@ -241,7 +241,7 @@ static void NewRailVehicleWndProc(Window *w, WindowEvent *e)
WP(w,buildtrain_d).sel_engine = selected_id;
- if (selected_id != -1) {
+ if (selected_id != INVALID_ENGINE) {
const RailVehicleInfo *rvi = RailVehInfo(selected_id);
if (!(rvi->flags & RVI_WAGON)) {
@@ -265,14 +265,13 @@ static void NewRailVehicleWndProc(Window *w, WindowEvent *e)
}
} break;
case 5: {
- int sel_eng;
- sel_eng = WP(w,buildtrain_d).sel_engine;
- if (sel_eng != -1)
+ EngineID sel_eng = WP(w,buildtrain_d).sel_engine;
+ if (sel_eng != INVALID_ENGINE)
DoCommandP(w->window_number, sel_eng, 0, (RailVehInfo(sel_eng)->flags & RVI_WAGON) ? CcBuildWagon : CcBuildLoco, CMD_BUILD_RAIL_VEHICLE | CMD_MSG(STR_882B_CAN_T_BUILD_RAILROAD_VEHICLE));
} break;
case 6: { /* rename */
- int sel_eng = WP(w,buildtrain_d).sel_engine;
- if (sel_eng != -1) {
+ EngineID sel_eng = WP(w,buildtrain_d).sel_engine;
+ if (sel_eng != INVALID_ENGINE) {
WP(w,buildtrain_d).rename_engine = sel_eng;
ShowQueryString(GetCustomEngineName(sel_eng),
STR_886A_RENAME_TRAIN_VEHICLE_TYPE, 31, 160, w->window_class, w->window_number);
diff --git a/vehicle.c b/vehicle.c
index 39a251a4c..266f9203f 100644
--- a/vehicle.c
+++ b/vehicle.c
@@ -334,7 +334,7 @@ void UpdateVehiclePosHash(Vehicle *v, int x, int y)
/* remove from hash table? */
if (old_hash != NULL) {
Vehicle *last = NULL;
- int idx = *old_hash;
+ VehicleID idx = *old_hash;
while ((u = GetVehicle(idx)) != v) {
idx = u->next_hash;
assert(idx != INVALID_VEHICLE);
@@ -1458,7 +1458,7 @@ int32 ReplaceVehicle(Vehicle *v)
(that is needed, because this CMD is called automaticly) */
if ( p->money64 < (int32)(p->engine_renew_money + build_cost + rear_engine_cost - v->value)) {
if (( _local_player == v->owner ) && ( v->unitnumber != 0 )) { //v->unitnumber = 0 for train cars
- int message;
+ StringID message;
SetDParam(0, v->unitnumber);
switch (v->type) {
case VEH_Train: message = STR_TRAIN_AUTORENEW_FAILED; break;
diff --git a/window.h b/window.h
index 79b8db305..8890777c4 100644
--- a/window.h
+++ b/window.h
@@ -352,8 +352,8 @@ assert_compile(WINDOW_CUSTOM_SIZE >= sizeof(tooltips_d));
typedef struct {
byte railtype;
byte sel_index;
- int16 sel_engine;
- int16 rename_engine;
+ EngineID sel_engine;
+ EngineID rename_engine;
} buildtrain_d;
assert_compile(WINDOW_CUSTOM_SIZE >= sizeof(buildtrain_d));