cpp/openttd-patchpack/source Changeset - r2498:8dfa040ed505

Changeset - r2498:8dfa040ed505

Parent rev.

Child rev.

[Not reviewed]

master

0 22 0

tron - 19 years ago 2005-10-07 07:35:15
tron@openttd.org

(svn r3024) -Codechange: Another batch of replacements of int/uint/int16/byte/-1 with proper types and constants

22 files changed with 84 insertions and 85 deletions:

aircraft_gui.c

economy.c

industry_cmd.c

landscape.c

misc_gui.c

network_data.h

news_gui.c

oldloader.c

openttd.c

player_gui.c

players.c

roadveh_gui.c

ship_gui.c

signs.h

station.h

station_cmd.c

station_gui.c

town.h

town_cmd.c

train_gui.c

vehicle.c

window.h

0 comments (0 inline, 0 general)

aircraft_gui.c

➞

Show inline comments

@@ @@ -126,13 +126,13 @@ static void NewAircraftWndProc(Window *w @@
 			const Engine* e = GetEngine(AIRCRAFT_ENGINES_INDEX);
 			int x = 2;
 			int y = 15;
 			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)) {
 					if (sel==0) selected_id = engine_id;
 					if (IS_INT_INSIDE(--pos, -w->vscroll.cap, 0)) {
 						DrawString(x+62, y+7, GetCustomEngineName(engine_id), sel==0 ? 0xC : 0x10);
@@ @@ -142,13 +142,13 @@ static void NewAircraftWndProc(Window *w @@
 					sel--;
+				}
 			} while (++engine_id, ++e,--num);
 			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);
+			}
+		}
 	} break;
 	case WE_CLICK:
@@ @@ -159,20 +159,20 @@ static void NewAircraftWndProc(Window *w @@
 				WP(w,buildtrain_d).sel_index = i + w->vscroll.pos;
 				SetWindowDirty(w);
+			}
 		} 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);
+			}
 		} break;
+		}

economy.c

➞

Show inline comments

@@ @@ -1320,13 +1320,13 @@ int LoadUnloadVehicle(Vehicle *v) @@
 	int result = 0;
 	uint16 last_visited;
 	Station *st;
 	GoodsEntry *ge;
 	int t;
 	uint count, cap;
-	byte old_player;
+	PlayerID old_player;
 	bool completely_empty = true;
 	assert(v->current_order.type == OT_LOADING);
 	v->cur_speed = 0;
@@ @@ -1557,14 +1557,14 @@ int32 CmdBuyShareInCompany(int x, int y, @@
 	/* We can not buy out a real player (temporarily). TODO: well, enable it obviously */
 	if (GetAmountOwnedBy(p, OWNER_SPECTATOR) == 1 && !p->is_ai) return 0;
 	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;
 		for (i = 0; p->share_owners[i] == _current_player;) {
 			if (++i == 4) {
@@ @@ -1599,13 +1599,13 @@ int32 CmdSellShareInCompany(int x, int y @@
 	/* adjust it a little to make it less profitable to sell and buy */
 	cost = CalculateCompanyValue(p) >> 2;
 	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);
+	}
 	return cost;
+}

industry_cmd.c

➞

Show inline comments

@@ @@ -1039,13 +1039,13 @@ static void ChopLumberMillTrees(Industry @@
 	for(a=1; a!=41; a+=2) {
 		for(dir=0; dir!=4; dir++) {
 			j = a;
 			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;
 					_industry_sound_ctr = 1;
 					_industry_sound_tile = tile;
 					SndPlayTileFx(SND_38_CHAINSAW, tile);
@@ @@ -1659,13 +1659,13 @@ static void PlaceInitialIndustry(byte ty @@
 	} else {
 		num = ScaleByMapSize(num);
+	}
 	if (_opt.diff.number_industries != 0)
+	{
-		byte old_player = _current_player;
+		PlayerID old_player = _current_player;
 		_current_player = OWNER_NONE;
 		assert(num > 0);
 		do {
 			int i = 2000;
 			do {
@@ @@ -1899,13 +1899,13 @@ static void ChangeIndustryProduction(Ind @@
+	}
+}
 void IndustryMonthlyLoop(void)
+{
 	Industry *i;
-	byte old_player = _current_player;
+	PlayerID old_player = _current_player;
 	_current_player = OWNER_NONE;
 	FOR_ALL_INDUSTRIES(i) {
 		if (i->xy != 0)
 			UpdateIndustryStatistics(i);
+	}

landscape.c

➞

Show inline comments

@@ @@ -377,13 +377,13 @@ void CDECL ModifyTile(TileIndex tile, ui @@
 		int x = 0;
 		if (flags & MP_MAP3HI) x = va_arg(va, int);
 		_m[tile].m4 = x;
+	}
 	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;
+	}
 	if (flags & MP_MAP5) {
 		_m[tile].m5 = va_arg(va, int);

misc_gui.c

➞

Show inline comments

@@ @@ -580,25 +580,25 @@ void ShowErrorMessage(StringID msg_1, St @@
 	w->desc_flags = WDF_STD_BTN | WDF_DEF_WIDGET;
+}
 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;
 	if (cost < 0) {
 		cost = -cost;
 		msg = STR_0803_INCOME;
@@ @@ -709,13 +709,13 @@ static void DrawStationCoverageText(cons @@
 	DrawStringMultiLine(str_x, str_y, STR_SPEC_USERSTRING, 144);
+}
 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);
+	}
+}

network_data.h

➞

Show inline comments

@@ @@ -32,13 +32,13 @@ typedef struct Packet { @@
 	PacketSize pos;
 	byte buffer[SEND_MTU];
 } 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
 	TileIndex tile; /// tile command being executed on
 	char text[80];
 	uint32 frame;  /// the frame in which this packet is executed

news_gui.c

➞

Show inline comments

@@ @@ -370,13 +370,13 @@ static inline void SetNewsDisplayValue(b @@
+}
 // open up an own newspaper window for the news item
 static void ShowNewspaper(NewsItem *ni)
+{
 	Window *w;
-	int sound;
+	SoundFx sound;
 	int top;
 	ni->flags &= ~(NF_NOEXPIRE | NF_FORCE_BIG);
 	ni->duration = 555;
 	sound = _news_sounds[ni->type];
 	if (sound != 0)

oldloader.c

➞

Show inline comments

@@ @@ -786,13 +786,13 @@ static bool LoadOldIndustry(LoadgameStat @@
 		i->town = GetTown(REMAP_TOWN_IDX(_old_town_index));
+	}
 	return true;
+}
-static uint _current_player_id;
+static PlayerID _current_player_id;
 static uint16 _old_inaugurated_year;
 static int32 _old_yearly;
 static const OldChunks player_yearly_chunk[] = {
 	OCL_VAR(  OC_INT32,   1, &_old_yearly ),
 	OCL_END()

openttd.c

➞

Show inline comments

@@ @@ -864,13 +864,13 @@ void StateGameLoop(void) @@
 		CallLandscapeTick();
 		CallWindowTickEvent();
 		NewsLoop();
 	} 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();
 		IncreaseDate();
 		RunTileLoop();
 		CallVehicleTicks();

player_gui.c

➞

Show inline comments

@@ @@ -688,13 +688,13 @@ static const WindowDesc _other_player_co @@
 	PlayerCompanyWndProc
 };
 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;
+}

players.c

➞

Show inline comments

@@ @@ -193,13 +193,13 @@ void InvalidatePlayerWindows(const Playe @@
 	InvalidateWindow(WC_FINANCES, pid);
+}
 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;
 			return false;
+		}
+	}
@@ @@ -281,13 +281,13 @@ bool CheckOwnership(PlayerID owner) @@
 	GetNameOfOwner(owner, 0);
 	return false;
+}
 bool CheckTileOwnership(TileIndex tile)
+{
-	byte owner = GetTileOwner(tile);
+	PlayerID owner = GetTileOwner(tile);
 	assert(owner <= OWNER_WATER);
 	if (owner == _current_player)
 		return true;
 	_error_message = STR_013B_OWNED_BY;

roadveh_gui.c

➞

Show inline comments

@@ @@ -409,13 +409,13 @@ static void DrawNewRoadVehWindow(Window @@
 		const Engine* e = GetEngine(ROAD_ENGINES_INDEX);
 		int x = 1;
 		int y = 15;
 		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)) {
 				if (sel==0) selected_id = engine_id;
 				if (IS_INT_INSIDE(--pos, -w->vscroll.cap, 0)) {
 					DrawString(x+59, y+2, GetCustomEngineName(engine_id), sel==0 ? 0xC : 0x10);
@@ @@ -424,13 +424,13 @@ static void DrawNewRoadVehWindow(Window @@
+				}
 				sel--;
+			}
 		} 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);
+		}
+	}
+}
 void CcBuildRoadVeh(bool success, TileIndex tile, uint32 p1, uint32 p2)
@@ @@ -462,20 +462,20 @@ static void NewRoadVehWndProc(Window *w, @@
 				WP(w,buildtrain_d).sel_index = i + w->vscroll.pos;
 				SetWindowDirty(w);
+			}
 		} 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);
+			}
 		}	break;
+		}

ship_gui.c

➞

Show inline comments

@@ @@ -349,13 +349,13 @@ static void NewShipWndProc(Window *w, Wi @@
 			const Engine* e = GetEngine(SHIP_ENGINES_INDEX);
 			int x = 2;
 			int y = 15;
 			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)) {
 					if (sel==0) selected_id = engine_id;
 					if (IS_INT_INSIDE(--pos, -w->vscroll.cap, 0)) {
 						DrawString(x+75, y+7, GetCustomEngineName(engine_id), sel==0 ? 0xC : 0x10);
@@ @@ -365,13 +365,13 @@ static void NewShipWndProc(Window *w, Wi @@
 					sel--;
+				}
 			} while (++engine_id, ++e,--num);
 			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);
+			}
+		}
 		break;
 	case WE_CLICK:
@@ @@ -381,20 +381,20 @@ static void NewShipWndProc(Window *w, Wi @@
 			if (i < w->vscroll.cap) {
 				WP(w,buildtrain_d).sel_index = i + w->vscroll.pos;
 				SetWindowDirty(w);
+			}
 		} 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);
+			}
 		}	break;
+		}

signs.h

➞

Show inline comments

@@ @@ -8,13 +8,13 @@ @@
 typedef struct SignStruct {
 	StringID     str;
 	ViewportSign sign;
 	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;
 } SignStruct;
 extern MemoryPool _sign_pool;

station.h

➞

Show inline comments

@@ @@ -120,17 +120,17 @@ enum { @@
 	CA_AIR_LARGE = 5,
 	CA_DOCK = 5,
 	CA_AIR_METRO = 6,
 	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;
 extern MemoryPool _station_pool;

station_cmd.c

➞

Show inline comments

@@ @@ -176,35 +176,34 @@ static byte FindCatchmentRadius(Station @@
 	return ret;
+}
 #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!!
 				if (st->airport_type == AT_OILRIG && st->facilities == (FACIL_AIRPORT|FACIL_DOCK))
 					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;
 				return CHECK_STATIONS_ERR;
+			}
+		}
 	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)
+{
 	switch (v->type) {
 		case VEH_Train: 		return st->train_tile;
@@ @@ -421,13 +420,13 @@ static bool GenerateStationName(Station @@
 done:
 	st->string_id = found + STR_SV_STNAME;
 	return true;
+}
 #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;
 	FOR_ALL_STATIONS(st) {
 		if (st->xy != 0 && (owner == OWNER_SPECTATOR || st->owner == owner)) {
@@ @@ -752,13 +751,13 @@ static void DeleteStationIfEmpty(Station @@
+}
 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;
 	uint tileh;
 	uint z;
 	int allowed_z = -1;
@@ @@ -807,23 +806,23 @@ int32 CheckFlatLandBelow(TileIndex tile, @@
 		} else if (allowed_z != flat_z) {
 			_error_message = STR_0007_FLAT_LAND_REQUIRED;
 			return CMD_ERROR;
+		}
 		// 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;
+				}
+			}
 		} else {
 			ret = DoCommandByTile(tile_cur, 0, 0, flags, CMD_LANDSCAPE_CLEAR);
@@ @@ -949,13 +948,13 @@ static void GetStationLayout(byte *layou @@
 int32 CmdBuildRailroadStation(int x, int y, uint32 flags, uint32 p1, uint32 p2)
+{
 	Station *st;
 	TileIndex tile_org;
 	int w_org, h_org;
 	int32 cost, ret;
-	int est;
+	StationID est;
 	int plat_len, numtracks;
 	int direction;
 	uint finalvalues[3];
 	SET_EXPENSES_TYPE(EXPENSES_CONSTRUCTION);
@@ @@ -983,13 +982,13 @@ int32 CmdBuildRailroadStation(int x, int @@
 	// these values are those that will be stored in train_tile and station_platforms
 	finalvalues[0] = tile_org;
 	finalvalues[1] = w_org;
 	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;
 	cost = ret + (numtracks * _price.train_station_track + _price.train_station_length) * plat_len;
 	// Make sure there are no similar stations around us.
@@ @@ -1187,14 +1186,15 @@ int32 CmdRemoveFromRailroadStation(int x @@
 	return _price.remove_rail_station;
+}
 // 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));
 	len = 0;
 	dir = _m[tile].m5&1;
 	delta = dir ? TileDiffXY(0, 1) : TileDiffXY(1, 0);
@@ @@ -2134,13 +2134,13 @@ static void DrawTile_Station(TileInfo *t @@
 	byte railtype = GB(_m[ti->tile].m3, 0, 4);
 	const RailtypeInfo *rti = GetRailTypeInfo(railtype);
 	SpriteID offset;
 	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);
+	}
 	// don't show foundation for docks (docks are between 76 (0x4C) and 81 (0x51))
@@ @@ -2677,13 +2677,13 @@ void OnTick_Station(void) @@
 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;
 	int i;
 	FOR_ALL_STATIONS(st) {

station_gui.c

➞

Show inline comments

@@ @@ -109,13 +109,13 @@ static void GlobalSortStationList(void) @@
 	memset(_station_sort_dirty, true, sizeof(_station_sort_dirty));
 	_global_station_sort_dirty = false;
 	DEBUG(misc, 1) ("Resorting global station list...");
+}
-static void MakeSortedStationList(byte owner)
+static void MakeSortedStationList(PlayerID owner)
+{
 	SortStruct *firstelement;
 	uint32 n = 0;
 	if (owner == 0) { // first element starts at 0th element and has n elements as described above
 		firstelement = &_station_sort[0];
@@ @@ -134,31 +134,31 @@ static void MakeSortedStationList(byte o @@
+}
 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);
 			DrawWindowWidgets(w);
+		}
@@ @@ -172,18 +172,18 @@ static void PlayerStationsWndProc(Window @@
 			if (w->vscroll.count == 0) { // player has no stations
 				DrawString(xb, y, STR_304A_NONE, 0);
 				return;
+			}
 			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);
 				x = DrawString(xb, y, STR_3049_0, 0) + 5;
 				// show cargo waiting and station ratings
@@ @@ -207,13 +207,13 @@ static void PlayerStationsWndProc(Window @@
 			if (id_v >= w->vscroll.cap) { return;} // click out of bounds
 			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
 				if (id_v >= _num_station_sort[owner]) { return;} // click out of station bound
 				st = GetStation(_station_sort[id_v].index);
@@ @@ -313,17 +313,17 @@ static void DrawStationViewWindow(Window @@
 	Station *st;
 	int i;
 	int num;
 	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++) {
 		if ((st->goods[i].waiting_acceptance & 0xFFF) != 0) {
 			num++;
 			if (st->goods[i].enroute_from != station_id)
@@ @@ -530,13 +530,13 @@ static const WindowDesc _station_view_de @@
 	WC_STATION_VIEW,0,
 	WDF_STD_TOOLTIPS | WDF_STD_BTN | WDF_DEF_WIDGET | WDF_UNCLICK_BUTTONS | WDF_STICKY_BUTTON,
 	_station_view_widgets,
 	StationViewWndProc
 };
-void ShowStationViewWindow(int station)
+void ShowStationViewWindow(StationID station)
+{
 	Window *w;
 	byte color;
 	w = AllocateWindowDescFront(&_station_view_desc, station);
 	if (w) {

town.h

➞

Show inline comments

@@ @@ -29,13 +29,13 @@ struct Town { @@
 	// Sort index in listings
 	byte sort_index_obsolete;
 	// 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];
 	// Maximum amount of passengers and mail that can be transported.
 	uint32 max_pass;
 	uint32 max_mail;

town_cmd.c

➞

Show inline comments

@@ @@ -779,13 +779,13 @@ static int GenRandomRoadBits(void) @@
 // Returns true if a house was built, or no if the build failed.
 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},
 		{ 1,  1},
 		{ 1, -1},
 		{-1, -1},
@@ @@ -1575,13 +1575,13 @@ static void TownActionRoadRebuild(Town * @@
 		NEWS_FLAGS(NM_NORMAL, NF_TILE, NT_GENERAL, 0), t->xy, 0);
+}
 static bool DoBuildStatueOfCompany(TileIndex tile)
+{
 	TileInfo ti;
-	byte old;
+	PlayerID old;
 	int32 r;
 	FindLandscapeHeightByTile(&ti, tile);
 	if (ti.tileh != 0)
 		return false;

train_gui.c

➞

Show inline comments

@@ @@ -170,13 +170,13 @@ void CcCloneTrain(bool success, uint til @@
 	v = GetVehicle(_new_train_id);
 	ShowTrainViewWindow(v);
+}
 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;
 	for (i = 0; i < NUM_TRAIN_ENGINES; i++) {
 		const Engine *e = GetEngine(i);
 		const RailVehicleInfo *rvi = RailVehInfo(i);
@@ @@ -226,25 +226,25 @@ static void NewRailVehicleWndProc(Window @@
+		{
 			byte railtype = WP(w,buildtrain_d).railtype;
 			int sel = WP(w,buildtrain_d).sel_index;
 			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.
 			 * XXX - DO NOT EVER DO THIS EVER AGAIN! GRRR hacking in wagons as
 			 * engines to get more types.. Stays here until we have our own format
 			 * then it is exit!!! */
 			engine_drawing_loop(&x, &y, &pos, &sel, &selected_id, railtype, w->vscroll.cap, true); // True engines
 			engine_drawing_loop(&x, &y, &pos, &sel, &selected_id, railtype, w->vscroll.cap, false); // Feeble wagons
 			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)) {
 					/* it's an engine */
 					DrawTrainEnginePurchaseInfo(2, w->widget[4].top + 1,selected_id);
 				} else {
@@ @@ -262,20 +262,19 @@ static void NewRailVehicleWndProc(Window @@
 			if (i < w->vscroll.cap) {
 				WP(w,buildtrain_d).sel_index = i + w->vscroll.pos;
 				SetWindowDirty(w);
+			}
 		} 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);
+			}
 		} break;
+		}

vehicle.c

➞

Show inline comments

@@ @@ -331,13 +331,13 @@ void UpdateVehiclePosHash(Vehicle *v, in @@
 	if (old_hash == new_hash)
 		return;
 	/* 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);
 			last = u;
+		}
@@ @@ -1455,13 +1455,13 @@ int32 ReplaceVehicle(Vehicle *v) @@
+	}
 	/* Check if there is money for the upgrade.. if not, give a nice news-item
 	    (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;
 				case VEH_Road:     message = STR_ROADVEHICLE_AUTORENEW_FAILED; break;
 				case VEH_Ship:     message = STR_SHIP_AUTORENEW_FAILED;        break;
 				case VEH_Aircraft: message = STR_AIRCRAFT_AUTORENEW_FAILED;    break;

window.h

➞

Show inline comments

@@ @@ -349,14 +349,14 @@ typedef struct { @@
 } tooltips_d;
 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));
 typedef struct {
 	byte vehicletype;
 	byte sel_index[2];

0 comments (0 inline, 0 general)