cpp/openttd-patchpack/source Changeset - r28825:9fe6b1220bd7

Changeset - r28825:9fe6b1220bd7

Parent rev.

Child rev.

[Not reviewed]

master

0 1 0

Peter Nelson - 2 months ago 2024-02-27 18:18:57
peter1138@openttd.org

Codechange: Use range-for instead of indexed loop. (#12191)

1 file changed with 4 insertions and 8 deletions:

src/vehicle_cmd.cpp

0 comments (0 inline, 0 general)

src/vehicle_cmd.cpp

➞

Show inline comments

@@ @@ -670,9 +670,7 @@ CommandCost CmdMassStartStopVehicle(DoCo @@
 		BuildDepotVehicleList(vli.vtype, tile, &list, nullptr);
+	}
 	for (uint i = 0; i < list.size(); i++) {
 		const Vehicle *v = list[i];
 	for (const Vehicle *v : list) {
 		if (!!(v->vehstatus & VS_STOPPED) != do_start) continue;
 		if (!vehicle_list_window && !v->IsChainInDepot()) continue;
@@ @@ -705,8 +703,8 @@ CommandCost CmdDepotSellAllVehicles(DoCo @@
 	CommandCost last_error = CMD_ERROR;
 	bool had_success = false;
 	for (uint i = 0; i < list.size(); i++) {
 		CommandCost ret = Command<CMD_SELL_VEHICLE>::Do(flags, list[i]->index, true, false, INVALID_CLIENT_ID);
 	for (const Vehicle *v : list) {
 		CommandCost ret = Command<CMD_SELL_VEHICLE>::Do(flags, v->index, true, false, INVALID_CLIENT_ID);
 		if (ret.Succeeded()) {
 			cost.AddCost(ret);
 			had_success = true;
@@ @@ -736,9 +734,7 @@ CommandCost CmdDepotMassAutoReplace(DoCo @@
 	/* Get the list of vehicles in the depot */
 	BuildDepotVehicleList(vehicle_type, tile, &list, &list, true);
 	for (uint i = 0; i < list.size(); i++) {
 		const Vehicle *v = list[i];
 	for (const Vehicle *v : list) {
 		/* Ensure that the vehicle completely in the depot */
 		if (!v->IsChainInDepot()) continue;

0 comments (0 inline, 0 general)