cpp/openttd-patchpack/source Changeset - r24513:ba5551156e4e

Changeset - r24513:ba5551156e4e

Parent rev.

Child rev.

[Not reviewed]

master

0 11 0

glx22 - 3 years ago 2020-12-26 23:13:56
glx@openttd.org

Codechange: Replace FOR_VEHICLE_ORDERS with range-based for loops

11 files changed with 69 insertions and 48 deletions:

src/autoreplace_cmd.cpp

src/industry_cmd.cpp

src/order_backup.cpp

src/order_base.h

src/order_cmd.cpp

src/order_gui.cpp

src/saveload/afterload.cpp

src/script/api/script_vehiclelist.cpp

src/station_cmd.cpp

src/vehicle_base.h

src/vehiclelist.cpp

0 comments (0 inline, 0 general)

src/autoreplace_cmd.cpp

➞

Show inline comments

@@ @@ -179,9 +179,8 @@ static bool VerifyAutoreplaceRefitForOrd @@
 	CargoTypes union_refit_mask_a = GetUnionOfArticulatedRefitMasks(v->engine_type, false);
 	CargoTypes union_refit_mask_b = GetUnionOfArticulatedRefitMasks(engine_type, false);
 	const Order *o;
 	const Vehicle *u = (v->type == VEH_TRAIN) ? v->First() : v;
-	FOR_VEHICLE_ORDERS(u, o) {
+	for (const Order *o : u->Orders()) {
 		if (!o->IsRefit() || o->IsAutoRefit()) continue;
 		CargoID cargo_type = o->GetRefitCargo();

src/industry_cmd.cpp

➞

Show inline comments

@@ @@ -2609,8 +2609,7 @@ static int WhoCanServiceIndustry(Industr @@
 		 * We cannot check the first of shared orders only, since the first vehicle in such a chain
 		 * may have a different cargo type.
 		 */
 		const Order *o;
 		FOR_VEHICLE_ORDERS(v, o) {
 		for (const Order *o : v->Orders()) {
 			if (o->IsType(OT_GOTO_STATION) && !(o->GetUnloadType() & OUFB_TRANSFER)) {
 				/* Vehicle visits a station to load or unload */
 				Station *st = Station::Get(o->GetDestination());

src/order_backup.cpp

➞

Show inline comments

@@ @@ -56,8 +56,7 @@ OrderBackup::OrderBackup(const Vehicle * @@
 		Order **tail = &this->orders;
 		/* Count the number of orders */
 		const Order *order;
 		FOR_VEHICLE_ORDERS(v, order) {
 		for (const Order *order : v->Orders()) {
 			Order *copy = new Order();
 			copy->AssignOrder(*order);
 			*tail = copy;

src/order_base.h

➞

Show inline comments

@@ @@ -392,6 +392,4 @@ public: @@
 	void DebugCheckSanity() const;
 };
 #define FOR_VEHICLE_ORDERS(v, order) for (order = (v->orders.list == nullptr) ? nullptr : v->orders.list->GetFirstOrder(); order != nullptr; order = order->next)
 #endif /* ORDER_BASE_H */

src/order_cmd.cpp

➞

Show inline comments

@@ @@ -969,8 +969,7 @@ void InsertOrder(Vehicle *v, Order *new_ @@
 	/* As we insert an order, the order to skip to will be 'wrong'. */
 	VehicleOrderID cur_order_id = 0;
 	Order *order;
 	FOR_VEHICLE_ORDERS(v, order) {
 	for (Order *order : v->Orders()) {
 		if (order->IsType(OT_CONDITIONAL)) {
 			VehicleOrderID order_id = order->GetConditionSkipToOrder();
 			if (order_id >= sel_ord) {
@@ @@ -1090,8 +1089,7 @@ void DeleteOrder(Vehicle *v, VehicleOrde @@
 	/* As we delete an order, the order to skip to will be 'wrong'. */
 	VehicleOrderID cur_order_id = 0;
 	Order *order = nullptr;
 	FOR_VEHICLE_ORDERS(v, order) {
 	for (Order *order : v->Orders()) {
 		if (order->IsType(OT_CONDITIONAL)) {
 			VehicleOrderID order_id = order->GetConditionSkipToOrder();
 			if (order_id >= sel_ord) {
@@ @@ -1225,8 +1223,7 @@ CommandCost CmdMoveOrder(TileIndex tile, @@
+		}
 		/* As we move an order, the order to skip to will be 'wrong'. */
 		Order *order;
 		FOR_VEHICLE_ORDERS(v, order) {
 		for (Order *order : v->Orders()) {
 			if (order->IsType(OT_CONDITIONAL)) {
 				VehicleOrderID order_id = order->GetConditionSkipToOrder();
 				if (order_id == moving_order) {
@@ @@ -1560,9 +1557,7 @@ CommandCost CmdCloneOrder(TileIndex tile @@
 			/* Is the vehicle already in the shared list? */
 			if (src->FirstShared() == dst->FirstShared()) return CMD_ERROR;
 			const Order *order;
 			FOR_VEHICLE_ORDERS(src, order) {
 			for (const Order *order : src->Orders()) {
 				if (!OrderGoesToStation(dst, order)) continue;
 				/* Allow copying unreachable destinations if they were already unreachable for the source.
@@ @@ -1613,8 +1608,7 @@ CommandCost CmdCloneOrder(TileIndex tile @@
 			/* Trucks can't copy all the orders from busses (and visa versa),
 			 * and neither can helicopters and aircraft. */
 			const Order *order;
 			FOR_VEHICLE_ORDERS(src, order) {
 			for (const Order *order : src->Orders()) {
 				if (OrderGoesToStation(dst, order) &&
 						!CanVehicleUseStation(dst, Station::Get(order->GetDestination()))) {
 					return_cmd_error(STR_ERROR_CAN_T_COPY_SHARE_ORDER);
@@ @@ -1632,7 +1626,6 @@ CommandCost CmdCloneOrder(TileIndex tile @@
+			}
 			if (flags & DC_EXEC) {
 				const Order *order;
 				Order *first = nullptr;
 				Order **order_dst;
@@ @@ -1642,7 +1635,7 @@ CommandCost CmdCloneOrder(TileIndex tile @@
 				DeleteVehicleOrders(dst, true, dst->GetNumOrders() != src->GetNumOrders());
 				order_dst = &first;
-				FOR_VEHICLE_ORDERS(src, order) {
+				for (const Order *order : src->Orders()) {
 					*order_dst = new Order();
 					(*order_dst)->AssignOrder(*order);
 					order_dst = &(*order_dst)->next;
@@ @@ -1749,13 +1742,12 @@ void CheckOrders(const Vehicle *v) @@
 	/* Only check every 20 days, so that we don't flood the message log */
 	if (v->owner == _local_company && v->day_counter % 20 == 0) {
 		const Order *order;
 		StringID message = INVALID_STRING_ID;
 		/* Check the order list */
 		int n_st = 0;
-		FOR_VEHICLE_ORDERS(v, order) {
+		for (const Order *order : v->Orders()) {
 			/* Dummy order? */
 			if (order->IsType(OT_DUMMY)) {
 				message = STR_NEWS_VEHICLE_HAS_VOID_ORDER;
@@ @@ -1829,7 +1821,7 @@ void RemoveOrderFromAllVehicles(OrderTyp @@
 		/* Clear the order from the order-list */
 		int id = -1;
-		FOR_VEHICLE_ORDERS(v, order) {
+		for (Order *order : v->Orders()) {
 			id++;
 restart:
@@ @@ -1879,9 +1871,7 @@ restart: @@
  */
 bool Vehicle::HasDepotOrder() const
+{
 	const Order *order;
 	FOR_VEHICLE_ORDERS(this, order) {
 	for (const Order *order : this->Orders()) {
 		if (order->IsType(OT_GOTO_DEPOT)) return true;
+	}
@@ @@ -1940,9 +1930,7 @@ uint16 GetServiceIntervalClamped(uint in @@
  */
 static bool CheckForValidOrders(const Vehicle *v)
+{
 	const Order *order;
 	FOR_VEHICLE_ORDERS(v, order) {
 	for (const Order *order : v->Orders()) {
 		switch (order->GetType()) {
 			case OT_GOTO_STATION:
 			case OT_GOTO_DEPOT:

src/order_gui.cpp

➞

Show inline comments

@@ @@ -793,8 +793,7 @@ public: @@
 		if (_settings_client.gui.quick_goto && v->owner == _local_company) {
 			/* If there are less than 2 station, make Go To active. */
 			int station_orders = 0;
 			const Order *order;
 			FOR_VEHICLE_ORDERS(v, order) {
 			for(const Order *order : v->Orders()) {
 				if (order->IsType(OT_GOTO_STATION)) station_orders++;
+			}

src/saveload/afterload.cpp

➞

Show inline comments

@@ @@ -1733,8 +1733,7 @@ bool AfterLoadGame() @@
 			v->current_order.ConvertFromOldSavegame();
 			if (v->type == VEH_ROAD && v->IsPrimaryVehicle() && v->FirstShared() == v) {
 				Order* order;
 				FOR_VEHICLE_ORDERS(v, order) order->SetNonStopType(ONSF_NO_STOP_AT_INTERMEDIATE_STATIONS);
 				for (Order *order : v->Orders()) order->SetNonStopType(ONSF_NO_STOP_AT_INTERMEDIATE_STATIONS);
+			}
+		}
 	} else if (IsSavegameVersionBefore(SLV_94)) {

src/script/api/script_vehiclelist.cpp

➞

Show inline comments

@@ @@ -31,9 +31,7 @@ ScriptVehicleList_Station::ScriptVehicle @@
 	for (const Vehicle *v : Vehicle::Iterate()) {
 		if ((v->owner == ScriptObject::GetCompany() || ScriptObject::GetCompany() == OWNER_DEITY) && v->IsPrimaryVehicle()) {
 			const Order *order;
 			FOR_VEHICLE_ORDERS(v, order) {
 			for (const Order *order : v->Orders()) {
 				if ((order->IsType(OT_GOTO_STATION) || order->IsType(OT_GOTO_WAYPOINT)) && order->GetDestination() == station_id) {
 					this->AddItem(v->index);
 					break;
@@ @@ -81,9 +79,7 @@ ScriptVehicleList_Depot::ScriptVehicleLi @@
 	for (const Vehicle *v : Vehicle::Iterate()) {
 		if ((v->owner == ScriptObject::GetCompany() || ScriptObject::GetCompany() == OWNER_DEITY) && v->IsPrimaryVehicle() && v->type == type) {
 			const Order *order;
 			FOR_VEHICLE_ORDERS(v, order) {
 			for (const Order *order : v->Orders()) {
 				if (order->IsType(OT_GOTO_DEPOT) && order->GetDestination() == dest) {
 					this->AddItem(v->index);
 					break;

src/station_cmd.cpp

➞

Show inline comments

@@ @@ -2486,8 +2486,7 @@ bool HasStationInUse(StationID station, @@
+{
 	for (const Vehicle *v : Vehicle::Iterate()) {
 		if ((v->owner == company) == include_company) {
 			const Order *order;
 			FOR_VEHICLE_ORDERS(v, order) {
 			for (const Order *order : v->Orders()) {
 				if ((order->IsType(OT_GOTO_STATION) || order->IsType(OT_GOTO_WAYPOINT)) && order->GetDestination() == station) {
 					return true;
+				}

src/vehicle_base.h

➞

Show inline comments

@@ @@ -969,6 +969,55 @@ public: @@
 		return v;
+	}
 	/**
 	 * Iterator to iterate orders
 	 * Supports deletion of current order
 	 */
 	struct OrderIterator {
 		typedef Order value_type;
 		typedef Order* pointer;
 		typedef Order& reference;
 		typedef size_t difference_type;
 		typedef std::forward_iterator_tag iterator_category;
 		explicit OrderIterator(OrderList *list) : list(list), prev(nullptr)
+		{
 			this->order = (this->list == nullptr) ? nullptr : this->list->GetFirstOrder();
+		}
 		bool operator==(const OrderIterator &other) const { return this->order == other.order; }
 		bool operator!=(const OrderIterator &other) const { return !(*this == other); }
 		Order * operator*() const { return this->order; }
 		OrderIterator & operator++()
+		{
 			this->prev = (this->prev == nullptr) ? this->list->GetFirstOrder() : this->prev->next;
 			this->order = (this->prev == nullptr) ? nullptr : this->prev->next;
 			return *this;
+		}
 	private:
 		OrderList *list;
 		Order *order;
 		Order *prev;
 	};
 	/**
 	 * Iterable ensemble of orders
 	 */
 	struct IterateWrapper {
 		OrderList *list;
 		IterateWrapper(OrderList *list = nullptr) : list(list) {}
 		OrderIterator begin() { return OrderIterator(this->list); }
 		OrderIterator end() { return OrderIterator(nullptr); }
 		bool empty() { return this->begin() == this->end(); }
 	};
 	/**
 	 * Returns an iterable ensemble of orders of a vehicle
 	 * @return an iterable ensemble of orders of a vehicle
 	 */
 	IterateWrapper Orders() const { return IterateWrapper(this->orders.list); }
 };
 /**

src/vehiclelist.cpp

➞

Show inline comments

@@ @@ -118,9 +118,7 @@ bool GenerateVehicleSortList(VehicleList @@
 		case VL_STATION_LIST:
 			for (const Vehicle *v : Vehicle::Iterate()) {
 				if (v->type == vli.vtype && v->IsPrimaryVehicle()) {
 					const Order *order;
 					FOR_VEHICLE_ORDERS(v, order) {
 					for (const Order *order : v->Orders()) {
 						if ((order->IsType(OT_GOTO_STATION) || order->IsType(OT_GOTO_WAYPOINT) || order->IsType(OT_IMPLICIT))
 								&& order->GetDestination() == vli.index) {
 							list->push_back(v);
@@ @@ -165,9 +163,7 @@ bool GenerateVehicleSortList(VehicleList @@
 		case VL_DEPOT_LIST:
 			for (const Vehicle *v : Vehicle::Iterate()) {
 				if (v->type == vli.vtype && v->IsPrimaryVehicle()) {
 					const Order *order;
 					FOR_VEHICLE_ORDERS(v, order) {
 					for (const Order *order : v->Orders()) {
 						if (order->IsType(OT_GOTO_DEPOT) && !(order->GetDepotActionType() & ODATFB_NEAREST_DEPOT) && order->GetDestination() == vli.index) {
 							list->push_back(v);
 							break;

0 comments (0 inline, 0 general)