cpp/openttd-patchpack/source Changeset - r12221:ff582fb76dde

Changeset - r12221:ff582fb76dde

Parent rev.

Child rev.

[Not reviewed]

master

0 13 0

smatz - 15 years ago 2009-06-24 19:26:41
smatz@openttd.org

(svn r16646) -Codechange: rename GetDepotByTile() to Depot::GetByTile()

13 files changed with 21 insertions and 21 deletions:

src/ai/api/ai_order.cpp

src/depot.cpp

src/depot_base.h

src/depot_gui.cpp

src/order_gui.cpp

src/rail_cmd.cpp

src/road_cmd.cpp

src/roadveh_cmd.cpp

src/ship_cmd.cpp

src/train_cmd.cpp

src/vehicle_gui.cpp

src/water_cmd.cpp

src/yapf/yapf_road.cpp

0 comments (0 inline, 0 general)

src/ai/api/ai_order.cpp

➞

Show inline comments

@@ @@ -345,7 +345,7 @@ static const Order *ResolveOrder(Vehicle @@
 				order.MakeGoToDepot(::GetStationIndex(destination), odtf, onsf, odaf);
 			} else {
 				if (::IsTileType(destination, MP_STATION)) return false;
-				order.MakeGoToDepot(::GetDepotByTile(destination)->index, odtf, onsf, odaf);
+				order.MakeGoToDepot(::Depot::GetByTile(destination)->index, odtf, onsf, odaf);
+			}
 			break;
+		}

src/depot.cpp

➞

Show inline comments

@@ @@ -17,10 +17,10 @@ INSTANTIATE_POOL_METHODS(Depot) @@
 /**
  * Gets a depot from a tile
+ *
+ * @param tile tile with depot
  * @return Returns the depot if the tile had a depot, else it returns NULL
  */
-Depot *GetDepotByTile(TileIndex tile)
+/* static */ Depot *Depot::GetByTile(TileIndex tile)
+{
 	/* A ship depot is multiple tiles. The north most tile is
 	 * always the ->xy tile, so make sure we always look for

src/depot_base.h

➞

Show inline comments

@@ @@ -19,10 +19,10 @@ struct Depot : DepotPool::PoolItem<&_dep @@
 	Depot(TileIndex xy = INVALID_TILE) : xy(xy) {}
 	~Depot();
 	static Depot *GetByTile(TileIndex tile);
 };
 Depot *GetDepotByTile(TileIndex tile);
 #define FOR_ALL_DEPOTS_FROM(var, start) FOR_ALL_ITEMS_FROM(Depot, depot_index, var, start)
 #define FOR_ALL_DEPOTS(var) FOR_ALL_DEPOTS_FROM(var, 0)

src/depot_gui.cpp

➞

Show inline comments

@@ @@ -341,7 +341,7 @@ struct DepotWindow : Window { @@
 		if (this->type == VEH_AIRCRAFT) {
 			SetDParam(0, GetStationIndex(tile)); // Airport name
 		} else {
-			Depot *depot = GetDepotByTile(tile);
+			Depot *depot = Depot::GetByTile(tile);
 			assert(depot != NULL);
 			SetDParam(0, depot->town_index);
@@ @@ -815,7 +815,7 @@ struct DepotWindow : Window { @@
 					TileIndex tile = this->window_number;
 					byte vehtype = this->type;
-					SetDParam(0, (vehtype == VEH_AIRCRAFT) ? GetStationIndex(tile) : GetDepotByTile(tile)->town_index);
+					SetDParam(0, (vehtype == VEH_AIRCRAFT) ? GetStationIndex(tile) : Depot::GetByTile(tile)->town_index);
 					ShowQuery(
 						confirm_captions[vehtype],
 						STR_DEPOT_SELL_CONFIRMATION_TEXT,

src/order_gui.cpp

➞

Show inline comments

@@ @@ -300,7 +300,7 @@ static Order GetOrderCmdFromTile(const V @@
 			case MP_RAILWAY:
 				if (v->type == VEH_TRAIN && IsTileOwner(tile, _local_company)) {
 					if (IsRailDepot(tile)) {
-						order.MakeGoToDepot(GetDepotByTile(tile)->index, ODTFB_PART_OF_ORDERS,
+						order.MakeGoToDepot(Depot::GetByTile(tile)->index, ODTFB_PART_OF_ORDERS,
 								_settings_client.gui.new_nonstop ? ONSF_NO_STOP_AT_INTERMEDIATE_STATIONS : ONSF_STOP_EVERYWHERE);
 						if (_ctrl_pressed) order.SetDepotOrderType((OrderDepotTypeFlags)(order.GetDepotOrderType() ^ ODTFB_SERVICE));
 						return order;
@@ @@ -310,7 +310,7 @@ static Order GetOrderCmdFromTile(const V @@
 			case MP_ROAD:
 				if (IsRoadDepot(tile) && v->type == VEH_ROAD && IsTileOwner(tile, _local_company)) {
-					order.MakeGoToDepot(GetDepotByTile(tile)->index, ODTFB_PART_OF_ORDERS,
+					order.MakeGoToDepot(Depot::GetByTile(tile)->index, ODTFB_PART_OF_ORDERS,
 							_settings_client.gui.new_nonstop ? ONSF_NO_STOP_AT_INTERMEDIATE_STATIONS : ONSF_STOP_EVERYWHERE);
 					if (_ctrl_pressed) order.SetDepotOrderType((OrderDepotTypeFlags)(order.GetDepotOrderType() ^ ODTFB_SERVICE));
 					return order;
@@ @@ -331,7 +331,7 @@ static Order GetOrderCmdFromTile(const V @@
 				if (IsShipDepot(tile) && IsTileOwner(tile, _local_company)) {
 					TileIndex tile2 = GetOtherShipDepotTile(tile);
-					order.MakeGoToDepot(GetDepotByTile(tile < tile2 ? tile : tile2)->index, ODTFB_PART_OF_ORDERS, ONSF_STOP_EVERYWHERE);
+					order.MakeGoToDepot(Depot::GetByTile(tile < tile2 ? tile : tile2)->index, ODTFB_PART_OF_ORDERS, ONSF_STOP_EVERYWHERE);
 					if (_ctrl_pressed) order.SetDepotOrderType((OrderDepotTypeFlags)(order.GetDepotOrderType() ^ ODTFB_SERVICE));
 					return order;
+				}

src/rail_cmd.cpp

➞

Show inline comments

@@ @@ -1441,7 +1441,7 @@ static CommandCost RemoveTrainDepot(Tile @@
+		}
 		DoClearSquare(tile);
-		delete GetDepotByTile(tile);
+		delete Depot::GetByTile(tile);
 		AddSideToSignalBuffer(tile, dir, owner);
 		YapfNotifyTrackLayoutChange(tile, DiagDirToDiagTrack(dir));
 		if (v != NULL) TryPathReserve(v, true);

src/road_cmd.cpp

➞

Show inline comments

@@ @@ -882,7 +882,7 @@ static CommandCost RemoveRoadDepot(TileI @@
 	if (flags & DC_EXEC) {
 		DoClearSquare(tile);
-		delete GetDepotByTile(tile);
+		delete Depot::GetByTile(tile);
+	}
 	return CommandCost(EXPENSES_CONSTRUCTION, _price.remove_road_depot);

src/roadveh_cmd.cpp

➞

Show inline comments

@@ @@ -374,7 +374,7 @@ static const Depot *FindClosestRoadDepot @@
 			NPFFoundTargetData ftd = NPFRouteToDepotBreadthFirstTwoWay(v->tile, trackdir, false, v->tile, ReverseTrackdir(trackdir), false, TRANSPORT_ROAD, v->compatible_roadtypes, v->owner, INVALID_RAILTYPES, 0);
-			if (ftd.best_bird_dist == 0) return GetDepotByTile(ftd.node.tile); // Target found
+			if (ftd.best_bird_dist == 0) return Depot::GetByTile(ftd.node.tile); // Target found
 		} break;
 		default:
@@ @@ -389,7 +389,7 @@ static const Depot *FindClosestRoadDepot @@
 				FollowTrack(v->tile, PATHFIND_FLAGS_NONE, TRANSPORT_ROAD, v->compatible_roadtypes, d, EnumRoadSignalFindDepot, NULL, &rfdd);
+			}
-			if (rfdd.best_length != UINT_MAX) return GetDepotByTile(rfdd.tile);
+			if (rfdd.best_length != UINT_MAX) return Depot::GetByTile(rfdd.tile);
 		} break;
+	}

src/ship_cmd.cpp

➞

Show inline comments

@@ @@ -100,7 +100,7 @@ static const Depot *FindClosestShipDepot @@
 		Trackdir trackdir = v->GetVehicleTrackdir();
 		NPFFoundTargetData ftd = NPFRouteToDepotTrialError(v->tile, trackdir, false, TRANSPORT_WATER, 0, v->owner, INVALID_RAILTYPES);
-		if (ftd.best_bird_dist == 0) return GetDepotByTile(ftd.node.tile); // Found target
+		if (ftd.best_bird_dist == 0) return Depot::GetByTile(ftd.node.tile); // Found target
 		return NULL; // Did not find target
+	}

src/train_cmd.cpp

➞

Show inline comments

@@ @@ -2231,7 +2231,7 @@ bool Train::FindClosestDepot(TileIndex * @@
 	if (tfdd.best_length == UINT_MAX) return false;
 	if (location    != NULL) *location    = tfdd.tile;
-	if (destination != NULL) *destination = GetDepotByTile(tfdd.tile)->index;
+	if (destination != NULL) *destination = Depot::GetByTile(tfdd.tile)->index;
 	if (reverse     != NULL) *reverse     = tfdd.reverse;
 	return true;
@@ @@ -4487,7 +4487,7 @@ static void CheckIfTrainNeedsService(Tra @@
 		return;
+	}
-	const Depot *depot = GetDepotByTile(tfdd.tile);
+	const Depot *depot = Depot::GetByTile(tfdd.tile);
 	if (v->current_order.IsType(OT_GOTO_DEPOT) &&
 			v->current_order.GetDestination() != depot->index &&

src/vehicle_gui.cpp

➞

Show inline comments

@@ @@ -1243,7 +1243,7 @@ void ShowVehicleListWindow(CompanyID com @@
 	if (vehicle_type == VEH_AIRCRAFT) {
 		depot_airport_index = GetStationIndex(depot_tile);
 	} else {
-		Depot *depot = GetDepotByTile(depot_tile);
+		Depot *depot = Depot::GetByTile(depot_tile);
 		if (depot == NULL) return; // no depot to show
 		depot_airport_index = depot->index;
+	}

src/water_cmd.cpp

➞

Show inline comments

@@ @@ -177,7 +177,7 @@ static CommandCost RemoveShipDepot(TileI @@
 	if (flags & DC_EXEC) {
 		/* Kill the depot, which is registered at the northernmost tile. Use that one */
-		delete GetDepotByTile(tile2 < tile ? tile2 : tile);
+		delete Depot::GetByTile(tile2 < tile ? tile2 : tile);
 		MakeWaterKeepingClass(tile,  GetTileOwner(tile));
 		MakeWaterKeepingClass(tile2, GetTileOwner(tile2));

src/yapf/yapf_road.cpp

➞

Show inline comments

@@ @@ -403,7 +403,7 @@ public: @@
 		Node *n = Yapf().GetBestNode();
 		TileIndex depot_tile = n->m_segment_last_tile;
 		assert(IsRoadDepotTile(depot_tile));
-		Depot *ret = GetDepotByTile(depot_tile);
+		Depot *ret = Depot::GetByTile(depot_tile);
 		return ret;
+	}
 };
@@ @@ -478,7 +478,7 @@ Depot *YapfFindNearestRoadDepot(const Ve @@
 	/* handle the case when our vehicle is already in the depot tile */
 	if (IsRoadDepotTile(tile)) {
 		/* only what we need to return is the Depot* */
-		return GetDepotByTile(tile);
+		return Depot::GetByTile(tile);
+	}
 	/* default is YAPF type 2 */

0 comments (0 inline, 0 general)