cpp/openttd-patchpack/source Changeset - r5665:f732864744b6

Changeset - r5665:f732864744b6

Parent rev.

Child rev.

[Not reviewed]

master

0 6 1

KUDr - 18 years ago 2007-01-14 19:18:50
kudr@openttd.org

(svn r8125) -Codechange: Station is now constructed/destroyed using new/delete operators (don't worry, they still use the same memory pool). Few station related functions turned into Station::methods (just first step). All this new stuff moved from station_cmd.cpp to the new file (station.cpp).

7 files changed with 296 insertions and 162 deletions:

projects/openttd.vcproj

projects/openttd_vs80.vcproj

source.list

src/economy.cpp

src/station.cpp

180

src/station.h

src/station_cmd.cpp

152

0 comments (0 inline, 0 general)

projects/openttd.vcproj

➞

Show inline comments

@@ @@ -341,6 +341,9 @@ @@
 				RelativePath=".\..\src\spritecache.cpp">
 			</File>
 			<File
 				RelativePath=".\..\src\station.cpp">
 			</File>
 			<File
 				RelativePath=".\..\src\string.cpp">
 			</File>
 			<File

projects/openttd_vs80.vcproj

➞

Show inline comments

@@ @@ -624,6 +624,9 @@ @@
 				RelativePath=".\..\src\spritecache.cpp">
 			</File>
 			<File
 				RelativePath=".\..\src\station.cpp">
 			</File>
 			<File
 				RelativePath=".\..\src\string.cpp">
 			</File>
 			<File

source.list

➞

Show inline comments

@@ @@ -63,6 +63,7 @@ settings.cpp @@
 signs.cpp
 sound.cpp
 spritecache.cpp
 station.cpp
 string.cpp
 strings.cpp
 texteff.cpp

src/economy.cpp

➞

Show inline comments

@@ @@ -1526,7 +1526,7 @@ int LoadUnloadVehicle(Vehicle *v, bool j @@
 	if (result != 0) {
 		InvalidateWindow(WC_VEHICLE_DETAILS, v->index);
-		MarkStationTilesDirty(st);
+		st->MarkTilesDirty();
 		if (result & 2) InvalidateWindow(WC_STATION_VIEW, last_visited);

src/station.cpp

➞

Show inline comments

@@ new file 100644 @@
 /* $Id$ */
 /** @file station_cmd.c */
 #include "stdafx.h"
 #include "openttd.h"
 #include "bridge_map.h"
 #include "debug.h"
 #include "functions.h"
 #include "station_map.h"
 #include "table/sprites.h"
 #include "table/strings.h"
 #include "map.h"
 #include "tile.h"
 #include "station.h"
 #include "gfx.h"
 #include "window.h"
 #include "viewport.h"
 #include "command.h"
 #include "town.h"
 #include "vehicle.h"
 #include "news.h"
 #include "saveload.h"
 #include "economy.h"
 #include "player.h"
 #include "airport.h"
 #include "sprite.h"
 #include "depot.h"
 #include "train.h"
 #include "water_map.h"
 #include "industry_map.h"
 #include "newgrf_callbacks.h"
 #include "newgrf_station.h"
 #include "yapf/yapf.h"
 #include "date.h"
 #include "helpers.hpp"
 void StationRect_Init(Station *st); // don't worry, will be removed soon
 Station::Station(TileIndex tile)
+{
 	DEBUG(station, cDebugCtorLevel, "I+%3d", index);
 	xy = tile;
 	airport_tile = dock_tile = train_tile = 0;
 	bus_stops = truck_stops = NULL;
 	had_vehicle_of_type = 0;
 	time_since_load = 255;
 	time_since_unload = 255;
 	delete_ctr = 0;
 	facilities = 0;
 	last_vehicle_type = VEH_Invalid;
 	for (GoodsEntry *ge = goods; ge != endof(goods); ge++) {
 		ge->waiting_acceptance = 0;
 		ge->days_since_pickup = 0;
 		ge->enroute_from = INVALID_STATION;
 		ge->rating = 175;
 		ge->last_speed = 0;
 		ge->last_age = 0xFF;
 		ge->feeder_profit = 0;
+	}
 	random_bits = Random();
 	waiting_triggers = 0;
 	StationRect_Init(this);
+}
 /**
 * Clean up a station by clearing vehicle orders and invalidating windows.
 * Aircraft-Hangar orders need special treatment here, as the hangars are
 * actually part of a station (tiletype is STATION), but the order type
 * is OT_GOTO_DEPOT.
 * @param st Station to be deleted
 */
 Station::~Station()
+{
 	DEBUG(station, cDebugCtorLevel, "I-%3d", index);
 	DeleteName(string_id);
 	MarkDirty();
 	RebuildStationLists();
 	InvalidateWindowClasses(WC_STATION_LIST);
 	DeleteWindowById(WC_STATION_VIEW, index);
 	/* Now delete all orders that go to the station */
 	RemoveOrderFromAllVehicles(OT_GOTO_STATION, index);
 	//Subsidies need removal as well
 	DeleteSubsidyWithStation(index);
 	free(speclist);
 	xy = 0;
+}
 void* Station::operator new(size_t size)
+{
 	Station *st = AllocateRaw();
 	return st;
+}
 void* Station::operator new(size_t size, int st_idx)
+{
 	if (!AddBlockIfNeeded(&_Station_pool, st_idx))
 		error("Stations: failed loading savegame: too many stations");
 	Station *st = GetStation(st_idx);
 	return st;
+}
 void Station::operator delete(void *p)
+{
+}
 void Station::operator delete(void *p, int st_idx)
+{
+}
 void Station::MarkDirty() const
+{
 	if (sign.width_1 != 0) {
 		InvalidateWindowWidget(WC_STATION_VIEW, index, 1);
 		MarkAllViewportsDirty(
 			sign.left - 6,
 			sign.top,
 			sign.left + (sign.width_1 << 2) + 12,
 			sign.top + 48);
+	}
+}
 void Station::MarkTilesDirty() const
+{
 	TileIndex tile = train_tile;
 	int w, h;
 	// XXX No station is recorded as 0, not INVALID_TILE...
 	if (tile == 0) return;
 	for (h = 0; h < trainst_h; h++) {
 		for (w = 0; w < trainst_w; w++) {
 			if (TileBelongsToRailStation(tile)) {
 				MarkTileDirtyByTile(tile);
+			}
 			tile += TileDiffXY(1, 0);
+		}
 		tile += TileDiffXY(-w, 1);
+	}
+}
 bool Station::TileBelongsToRailStation(TileIndex tile) const
+{
 	return IsTileType(tile, MP_STATION) && GetStationIndex(tile) == index && IsRailwayStation(tile);
+}
 /*static*/ Station *Station::AllocateRaw(void)
+{
 	Station *st = NULL;
 	/* We don't use FOR_ALL here, because FOR_ALL skips invalid items.
 	 * TODO - This is just a temporary stage, this will be removed. */
 	for (st = GetStation(0); st != NULL; st = (st->index + 1U < GetStationPoolSize()) ? GetStation(st->index + 1U) : NULL) {
 		if (!IsValidStation(st)) {
 			StationID index = st->index;
 			memset(st, 0, sizeof(Station));
 			st->index = index;
 			return st;
+		}
+	}
 	/* Check if we can add a block to the pool */
 	if (AddBlockToPool(&_Station_pool)) return AllocateRaw();
 	_error_message = STR_3008_TOO_MANY_STATIONS_LOADING;
 	return NULL;
+}

src/station.h

➞

Show inline comments

@@ @@ -3,6 +3,7 @@ @@
 #ifndef STATION_H
 #define STATION_H
 #include <deque>
 #include "player.h"
 #include "oldpool.h"
 #include "sprite.h"
@@ @@ -97,6 +98,26 @@ struct Station { @@
 	byte blocked_months_obsolete;
 	Rect rect; ///< Station spread out rectangle (not saved) maintained by StationRect_xxx() functions
 	static const int cDebugCtorLevel = 1;
 	Station(TileIndex tile = 0);
 	~Station();
 	/* normal new/delete operators. Used when building/removing station */
 	void* operator new (size_t size);
 	void operator delete(void *p);
 	/* new/delete operators accepting station index. Used when loading station from savegame. */
 	void* operator new (size_t size, int st_idx);
 	void operator delete(void *p, int st_idx);
 	void MarkDirty() const;
 	void MarkTilesDirty() const;
 	bool TileBelongsToRailStation(TileIndex tile) const;
 protected:
 	static Station *AllocateRaw(void);
 };
 enum {
@@ @@ -176,14 +197,6 @@ static inline bool IsValidStationID(Stat @@
 	return index < GetStationPoolSize() && IsValidStation(GetStation(index));
+}
 void DestroyStation(Station *st);
 static inline void DeleteStation(Station *st)
+{
 	DestroyStation(st);
 	st->xy = 0;
+}
 #define FOR_ALL_STATIONS_FROM(st, start) for (st = GetStation(start); st != NULL; st = (st->index + 1U < GetStationPoolSize()) ? GetStation(st->index + 1U) : NULL) if (IsValidStation(st))
 #define FOR_ALL_STATIONS(st) FOR_ALL_STATIONS_FROM(st, 0)
@@ @@ -219,7 +232,6 @@ void GetProductionAroundTiles(AcceptedCa @@
 void GetAcceptanceAroundTiles(AcceptedCargo accepts, TileIndex tile, int w, int h, int rad);
 uint GetStationPlatforms(const Station *st, TileIndex tile);
 uint GetPlatformLength(TileIndex tile, DiagDirection dir);
 void MarkStationTilesDirty(const Station *st);
 const DrawTileSprites *GetStationTileLayout(byte gfx);

src/station_cmd.cpp

➞

Show inline comments

@@ @@ -42,7 +42,7 @@ typedef enum StationRectModes @@
 	RECT_MODE_FORCE
 } StationRectMode;
-static void StationRect_Init(Station *st);
 void StationRect_Init(Station *st);
 static bool StationRect_IsEmpty(Station *st);
 static bool StationRect_BeforeAddTile(Station *st, TileIndex tile, StationRectMode mode);
 static bool StationRect_BeforeAddRect(Station *st, TileIndex tile, int w, int h, StationRectMode mode);
@@ @@ -70,6 +70,7 @@ static void StationPoolCleanBlock(uint s @@
 		Station *st = GetStation(i);
 		free(st->speclist);
 		st->speclist = NULL;
 		if (IsValidStation(st)) st->~Station();
+	}
+}
@@ @@ -91,43 +92,6 @@ DEFINE_OLD_POOL(RoadStop, RoadStop, Road @@
 extern void UpdateAirplanesOnNewStation(Station *st);
 static bool TileBelongsToRailStation(const Station *st, TileIndex tile)
+{
 	return IsTileType(tile, MP_STATION) && GetStationIndex(tile) == st->index && IsRailwayStation(tile);
+}
 void MarkStationTilesDirty(const Station *st)
+{
 	TileIndex tile = st->train_tile;
 	int w, h;
 	// XXX No station is recorded as 0, not INVALID_TILE...
 	if (tile == 0) return;
 	for (h = 0; h < st->trainst_h; h++) {
 		for (w = 0; w < st->trainst_w; w++) {
 			if (TileBelongsToRailStation(st, tile)) {
 				MarkTileDirtyByTile(tile);
+			}
 			tile += TileDiffXY(1, 0);
+		}
 		tile += TileDiffXY(-w, 1);
+	}
+}
 static void MarkStationDirty(const Station* st)
+{
 	if (st->sign.width_1 != 0) {
 		InvalidateWindowWidget(WC_STATION_VIEW, st->index, 1);
 		MarkAllViewportsDirty(
 			st->sign.left - 6,
 			st->sign.top,
 			st->sign.left + (st->sign.width_1 << 2) + 12,
 			st->sign.top + 48);
+	}
+}
 static void InitializeRoadStop(RoadStop *road_stop, RoadStop *previous, TileIndex tile, StationID index)
+{
 	road_stop->xy = tile;
@@ @@ -263,7 +227,6 @@ static Station *AllocateStation(void) @@
 			memset(st, 0, sizeof(Station));
 			st->index = index;
 			return st;
+		}
+	}
@@ @@ -456,37 +419,6 @@ static Station* GetClosestStationFromTil @@
 	return best_station;
+}
 static void StationInitialize(Station *st, TileIndex tile)
+{
 	GoodsEntry *ge;
 	st->xy = tile;
 	st->airport_tile = st->dock_tile = st->train_tile = 0;
 	st->bus_stops = st->truck_stops = NULL;
 	st->had_vehicle_of_type = 0;
 	st->time_since_load = 255;
 	st->time_since_unload = 255;
 	st->delete_ctr = 0;
 	st->facilities = 0;
 	st->last_vehicle_type = VEH_Invalid;
 	for (ge = st->goods; ge != endof(st->goods); ge++) {
 		ge->waiting_acceptance = 0;
 		ge->days_since_pickup = 0;
 		ge->enroute_from = INVALID_STATION;
 		ge->rating = 175;
 		ge->last_speed = 0;
 		ge->last_age = 0xFF;
 		ge->feeder_profit = 0;
+	}
 	st->random_bits = Random();
 	st->waiting_triggers = 0;
 	StationRect_Init(st);
+}
 // Update the virtual coords needed to draw the station sign.
 // st = Station to update for.
 static void UpdateStationVirtCoord(Station *st)
@@ @@ -514,9 +446,9 @@ void UpdateAllStationVirtCoord(void) @@
 // Update the station virt coords while making the modified parts dirty.
 static void UpdateStationVirtCoordDirty(Station *st)
+{
-	MarkStationDirty(st);
+	st->MarkDirty();
 	UpdateStationVirtCoord(st);
-	MarkStationDirty(st);
+	st->MarkDirty();
+}
 // Get a mask of the cargo types that the station accepts.
@@ @@ -1020,6 +952,10 @@ int32 CmdBuildRailroadStation(TileIndex @@
 		if (st != NULL && st->facilities) st = NULL;
+	}
 	/* In case of new station if DC_EXEC is NOT set we still need to create the station
 	 * to test if everything is OK. In this case we need to delete it before return. */
 	std::auto_ptr<Station> st_auto_delete;
 	if (st != NULL) {
 		// Reuse an existing station.
 		if (st->owner != OWNER_NONE && st->owner != _current_player)
@@ @@ -1036,17 +972,19 @@ int32 CmdBuildRailroadStation(TileIndex @@
 		//XXX can't we pack this in the "else" part of the if above?
 		if (!StationRect_BeforeAddRect(st, tile_org, w_org, h_org, RECT_MODE_TEST)) return CMD_ERROR;
 	} else {
 		// Create a new station
 		st = AllocateStation();
 		/* allocate and initialize new station */
 		st = new Station(tile_org);
 		if (st == NULL) return CMD_ERROR;
 		/* ensure that in case of error (or no DC_EXEC) the station gets deleted upon return */
 		st_auto_delete = std::auto_ptr<Station>(st);
 		st->town = ClosestTownFromTile(tile_org, (uint)-1);
 		if (IsValidPlayer(_current_player) && (flags & DC_EXEC))
 		if (!GenerateStationName(st, tile_org, 0)) return CMD_ERROR;
 		if (IsValidPlayer(_current_player) && (flags & DC_EXEC) != 0) {
 			SETBIT(st->town->have_ratings, _current_player);
 		if (!GenerateStationName(st, tile_org, 0)) return CMD_ERROR;
 		if (flags & DC_EXEC) StationInitialize(st, tile_org);
+		}
+	}
 	/* Check if the given station class is valid */
@@ @@ -1126,11 +1064,13 @@ int32 CmdBuildRailroadStation(TileIndex @@
 			tile_org += tile_delta ^ TileDiffXY(1, 1); // perpendicular to tile_delta
 		} while (--numtracks);
-		MarkStationTilesDirty(st);
+		st->MarkTilesDirty();
 		UpdateStationVirtCoordDirty(st);
 		UpdateStationAcceptance(st, false);
 		RebuildStationLists();
 		InvalidateWindow(WC_STATION_LIST, st->owner);
 		/* success, so don't delete the new station */
 		st_auto_delete.release();
+	}
 	return cost;
@@ @@ -1148,7 +1088,7 @@ restart: @@
 	// too small?
 	if (w != 0 && h != 0) {
 		// check the left side, x = constant, y changes
-		for (i = 0; !TileBelongsToRailStation(st, tile + TileDiffXY(0, i));) {
+		for (i = 0; !st->TileBelongsToRailStation(tile + TileDiffXY(0, i));) {
 			// the left side is unused?
 			if (++i == h) {
 				tile += TileDiffXY(1, 0);
@@ @@ -1158,7 +1098,7 @@ restart: @@
+		}
 		// check the right side, x = constant, y changes
-		for (i = 0; !TileBelongsToRailStation(st, tile + TileDiffXY(w - 1, i));) {
+		for (i = 0; !st->TileBelongsToRailStation(tile + TileDiffXY(w - 1, i));) {
 			// the right side is unused?
 			if (++i == h) {
 				w--;
@@ @@ -1167,7 +1107,7 @@ restart: @@
+		}
 		// check the upper side, y = constant, x changes
-		for (i = 0; !TileBelongsToRailStation(st, tile + TileDiffXY(i, 0));) {
+		for (i = 0; !st->TileBelongsToRailStation(tile + TileDiffXY(i, 0));) {
 			// the left side is unused?
 			if (++i == w) {
 				tile += TileDiffXY(0, 1);
@@ @@ -1177,7 +1117,7 @@ restart: @@
+		}
 		// check the lower side, y = constant, x changes
-		for (i = 0; !TileBelongsToRailStation(st, tile + TileDiffXY(i, h - 1));) {
+		for (i = 0; !st->TileBelongsToRailStation(tile + TileDiffXY(i, h - 1));) {
 			// the left side is unused?
 			if (++i == w) {
 				h--;
@@ @@ -1224,7 +1164,7 @@ int32 CmdRemoveFromRailroadStation(TileI @@
 		// now we need to make the "spanned" area of the railway station smaller if we deleted something at the edges.
 		// we also need to adjust train_tile.
 		MakeRailwayStationAreaSmaller(st);
-		MarkStationTilesDirty(st);
+		st->MarkTilesDirty();
 		UpdateStationSignCoord(st);
 		// if we deleted the whole station, delete the train facility.
@@ @@ -1244,7 +1184,7 @@ uint GetStationPlatforms(const Station * @@
 	TileIndexDiff delta;
 	Axis axis;
 	uint len;
-	assert(TileBelongsToRailStation(st, tile));
+	assert(st->TileBelongsToRailStation(tile));
 	len = 0;
 	axis = GetRailStationAxis(tile);
@@ @@ -1255,14 +1195,14 @@ uint GetStationPlatforms(const Station * @@
 	do {
 		t -= delta;
 		len++;
-	} while (TileBelongsToRailStation(st, t) && GetRailStationAxis(t) == axis);
+	} while (st->TileBelongsToRailStation(t) && GetRailStationAxis(t) == axis);
 	// find ending tile
 	t = tile;
 	do {
 		t += delta;
 		len++;
-	} while (TileBelongsToRailStation(st, t) && GetRailStationAxis(t) == axis);
+	} while (st->TileBelongsToRailStation(t) && GetRailStationAxis(t) == axis);
 	return len - 1;
+}
@@ @@ -1314,7 +1254,7 @@ static int32 RemoveRailroadStation(Stati @@
 		int w_bak = w;
 		do {
 			// for nonuniform stations, only remove tiles that are actually train station tiles
-			if (TileBelongsToRailStation(st, tile)) {
+			if (st->TileBelongsToRailStation(tile)) {
 				if (!EnsureNoVehicle(tile))
 					return CMD_ERROR;
 				cost += _price.remove_rail_station;
@@ @@ -1456,6 +1396,10 @@ int32 CmdBuildRoadStop(TileIndex tile, u @@
 		return_cmd_error(type ? STR_3008B_TOO_MANY_TRUCK_STOPS : STR_3008A_TOO_MANY_BUS_STOPS);
+	}
 	/* In case of new station if DC_EXEC is NOT set we still need to create the station
 	* to test if everything is OK. In this case we need to delete it before return. */
 	std::auto_ptr<Station> st_auto_delete;
 	if (st != NULL) {
 		if (st->owner != OWNER_NONE && st->owner != _current_player) {
 			return_cmd_error(STR_3009_TOO_CLOSE_TO_ANOTHER_STATION);
@@ @@ -1465,24 +1409,24 @@ int32 CmdBuildRoadStop(TileIndex tile, u @@
 		FindRoadStopSpot(type, st, &currstop, &prev);
 	} else {
 		Town *t;
 		st = AllocateStation();
 		/* allocate and initialize new station */
 		st = new Station(tile);
 		if (st == NULL) return CMD_ERROR;
 		st->town = t = ClosestTownFromTile(tile, (uint)-1);
 		/* ensure that in case of error (or no DC_EXEC) the new station gets deleted upon return */
 		st_auto_delete = std::auto_ptr<Station>(st);
 		Town *t = st->town = ClosestTownFromTile(tile, (uint)-1);
 		if (!GenerateStationName(st, tile, 0)) return CMD_ERROR;
 		FindRoadStopSpot(type, st, &currstop, &prev);
 		if (IsValidPlayer(_current_player) && (flags & DC_EXEC)) {
+		if (IsValidPlayer(_current_player) && (flags & DC_EXEC) != 0) {
 			SETBIT(t->have_ratings, _current_player);
+		}
 		st->sign.width_1 = 0;
 		if (!GenerateStationName(st, tile, 0)) return CMD_ERROR;
 		if (flags & DC_EXEC) StationInitialize(st, tile);
+	}
 	cost += (type) ? _price.build_truck_station : _price.build_bus_station;
@@ @@ -1507,6 +1451,8 @@ int32 CmdBuildRoadStop(TileIndex tile, u @@
 		UpdateStationAcceptance(st, false);
 		RebuildStationLists();
 		InvalidateWindow(WC_STATION_LIST, st->owner);
 		/* success, so don't delete the new station */
 		st_auto_delete.release();
+	}
 	return cost;
+}
@@ @@ -1711,6 +1657,10 @@ int32 CmdBuildAirport(TileIndex tile, ui @@
 		return CMD_ERROR;
+	}
 	/* In case of new station if DC_EXEC is NOT set we still need to create the station
 	 * to test if everything is OK. In this case we need to delete it before return. */
 	std::auto_ptr<Station> st_auto_delete;
 	if (st != NULL) {
 		if (st->owner != OWNER_NONE && st->owner != _current_player)
 			return_cmd_error(STR_3009_TOO_CLOSE_TO_ANOTHER_STATION);
@@ @@ -1722,13 +1672,18 @@ int32 CmdBuildAirport(TileIndex tile, ui @@
 	} else {
 		airport_upgrade = false;
 		st = AllocateStation();
 		/* allocate and initialize new station */
 		st = new Station(tile);
 		if (st == NULL) return CMD_ERROR;
 		/* ensure that in case of error (or no DC_EXEC) the station gets deleted upon return */
 		st_auto_delete = std::auto_ptr<Station>(st);
 		st->town = t;
 		if (IsValidPlayer(_current_player) && (flags & DC_EXEC))
+		if (IsValidPlayer(_current_player) && (flags & DC_EXEC) != 0) {
 			SETBIT(t->have_ratings, _current_player);
+		}
 		st->sign.width_1 = 0;
@@ @@ -1736,8 +1691,6 @@ int32 CmdBuildAirport(TileIndex tile, ui @@
 		// type 5 name, which is heliport, otherwise airport names (1)
 		if (!GenerateStationName(st, tile, (p1 == AT_HELIPORT)||(p1 == AT_HELIDEPOT)||(p1 == AT_HELISTATION) ? 5 : 1))
 			return CMD_ERROR;
 		if (flags & DC_EXEC) StationInitialize(st, tile);
+	}
 	cost += _price.build_airport * w * h;
@@ @@ -1775,6 +1728,8 @@ int32 CmdBuildAirport(TileIndex tile, ui @@
 		UpdateStationAcceptance(st, false);
 		RebuildStationLists();
 		InvalidateWindow(WC_STATION_LIST, st->owner);
 		/* success, so don't delete the new station */
 		st_auto_delete.release();
+	}
 	return cost;
@@ @@ -1841,16 +1796,19 @@ int32 CmdBuildBuoy(TileIndex tile, uint3 @@
 	if (!IsClearWaterTile(tile) || tile == 0) return_cmd_error(STR_304B_SITE_UNSUITABLE);
 	st = AllocateStation();
 	/* allocate and initialize new station */
 	st = new Station(tile);
 	if (st == NULL) return CMD_ERROR;
 	/* ensure that in case of error (or no DC_EXEC) the station gets deleted upon return */
 	std::auto_ptr<Station> st_auto_delete = std::auto_ptr<Station>(st);
 	st->town = ClosestTownFromTile(tile, (uint)-1);
 	st->sign.width_1 = 0;
 	if (!GenerateStationName(st, tile, 4)) return CMD_ERROR;
 	if (flags & DC_EXEC) {
 		StationInitialize(st, tile);
 		st->dock_tile = tile;
 		st->facilities |= FACIL_DOCK;
 		/* Buoys are marked in the Station struct by this flag. Yes, it is this
@@ @@ -1866,6 +1824,8 @@ int32 CmdBuildBuoy(TileIndex tile, uint3 @@
 		UpdateStationAcceptance(st, false);
 		RebuildStationLists();
 		InvalidateWindow(WC_STATION_LIST, st->owner);
 		/* success, so don't delete the new station */
 		st_auto_delete.release();
+	}
 	return _price.build_dock;
@@ @@ -1983,6 +1943,10 @@ int32 CmdBuildDock(TileIndex tile, uint3 @@
 		if (st!=NULL && st->facilities) st = NULL;
+	}
 	/* In case of new station if DC_EXEC is NOT set we still need to create the station
 	* to test if everything is OK. In this case we need to delete it before return. */
 	std::auto_ptr<Station> st_auto_delete;
 	if (st != NULL) {
 		if (st->owner != OWNER_NONE && st->owner != _current_player)
 			return_cmd_error(STR_3009_TOO_CLOSE_TO_ANOTHER_STATION);
@@ @@ -1991,21 +1955,22 @@ int32 CmdBuildDock(TileIndex tile, uint3 @@
 		if (st->dock_tile != 0) return_cmd_error(STR_304C_TOO_CLOSE_TO_ANOTHER_DOCK);
 	} else {
 		Town *t;
 		st = AllocateStation();
 		/* allocate and initialize new station */
 		st = new Station(tile);
 		if (st == NULL) return CMD_ERROR;
 		st->town = t = ClosestTownFromTile(tile, (uint)-1);
 		if (IsValidPlayer(_current_player) && (flags & DC_EXEC))
 		/* ensure that in case of error (or no DC_EXEC) the station gets deleted upon return */
 		st_auto_delete = std::auto_ptr<Station>(st);
 		Town *t = st->town = ClosestTownFromTile(tile, (uint)-1);
 		if (IsValidPlayer(_current_player) && (flags & DC_EXEC) != 0) {
 			SETBIT(t->have_ratings, _current_player);
+		}
 		st->sign.width_1 = 0;
 		if (!GenerateStationName(st, tile, 3)) return CMD_ERROR;
 		if (flags & DC_EXEC) StationInitialize(st, tile);
+	}
 	if (flags & DC_EXEC) {
@@ @@ -2024,6 +1989,8 @@ int32 CmdBuildDock(TileIndex tile, uint3 @@
 		UpdateStationAcceptance(st, false);
 		RebuildStationLists();
 		InvalidateWindow(WC_STATION_LIST, st->owner);
 		/* success, so don't delete the new station */
 		st_auto_delete.release();
+	}
 	return _price.build_dock;
+}
@@ @@ -2402,41 +2369,13 @@ void DestroyRoadStop(RoadStop* rs) @@
 	if (rs->next != NULL) rs->next->prev = rs->prev;
+}
 /**
  * Clean up a station by clearing vehicle orders and invalidating windows.
  * Aircraft-Hangar orders need special treatment here, as the hangars are
  * actually part of a station (tiletype is STATION), but the order type
  * is OT_GOTO_DEPOT.
  * @param st Station to be deleted
  */
 void DestroyStation(Station *st)
+{
 	StationID index;
 	index = st->index;
 	DeleteName(st->string_id);
 	MarkStationDirty(st);
 	RebuildStationLists();
 	InvalidateWindowClasses(WC_STATION_LIST);
 	DeleteWindowById(WC_STATION_VIEW, index);
 	/* Now delete all orders that go to the station */
 	RemoveOrderFromAllVehicles(OT_GOTO_STATION, index);
 	//Subsidies need removal as well
 	DeleteSubsidyWithStation(index);
 	free(st->speclist);
+}
 void DeleteAllPlayerStations(void)
+{
 	Station *st;
 	FOR_ALL_STATIONS(st) {
-		if (IsValidPlayer(st->owner)) DeleteStation(st);
+		if (IsValidPlayer(st->owner)) delete st;
+	}
+}
@@ @@ -2445,7 +2384,7 @@ static void StationHandleBigTick(Station @@
+{
 	UpdateStationAcceptance(st, true);
-	if (st->facilities == 0 && ++st->delete_ctr >= 8) DeleteStation(st);
+	if (st->facilities == 0 && ++st->delete_ctr >= 8) delete st;
+}
@@ @@ -2609,7 +2548,7 @@ static void UpdateStationWaiting(Station @@
 	st->goods[type].enroute_time = 0;
 	st->goods[type].enroute_from = st->index;
 	InvalidateWindow(WC_STATION_VIEW, st->index);
-	MarkStationTilesDirty(st);
+	st->MarkTilesDirty();
+}
 /** Rename a station
@@ @@ -2786,7 +2725,7 @@ uint MoveGoodsToStation(TileIndex tile, @@
 void BuildOilRig(TileIndex tile)
+{
 	uint j;
-	Station *st = AllocateStation();
+	Station *st = new Station();
 	if (st == NULL) {
 		DEBUG(misc, 0, "Can't allocate station for oilrig at 0x%X, reverting to oilrig only", tile);
@@ @@ -2843,7 +2782,7 @@ void DeleteOilRig(TileIndex tile) @@
 	st->facilities &= ~(FACIL_AIRPORT | FACIL_DOCK);
 	st->airport_flags = 0;
 	UpdateStationVirtCoordDirty(st);
-	DeleteStation(st);
+	if (st->facilities == 0) delete st;
+}
 static void ChangeTileOwner_Station(TileIndex tile, PlayerID old_player, PlayerID new_player)
@@ @@ -3084,12 +3023,8 @@ static void Load_STNS(void) @@
+{
 	int index;
 	while ((index = SlIterateArray()) != -1) {
 		Station *st;
 		if (!AddBlockIfNeeded(&_Station_pool, index))
 			error("Stations: failed loading savegame: too many stations");
 		st = GetStation(index);
 		Station *st = new (index) Station();
 		SaveLoad_STNS(st);
 		// this means it's an oldstyle savegame without support for nonuniform stations
@@ @@ -3167,7 +3102,7 @@ static inline bool PtInRectXY(Rect *r, i @@
 	return (r->left <= x && x <= r->right && r->top <= y && y <= r->bottom);
+}
-static void StationRect_Init(Station *st)
 void StationRect_Init(Station *st)
+{
 	Rect *r = &st->rect;
 	r->left = r->top = r->right = r->bottom = 0;

0 comments (0 inline, 0 general)