diff --git a/src/articulated_vehicles.cpp b/src/articulated_vehicles.cpp index 8a67b2a9a1..824611ddcd 100644 --- a/src/articulated_vehicles.cpp +++ b/src/articulated_vehicles.cpp @@ -28,7 +28,7 @@ static const uint MAX_ARTICULATED_PARTS = 100; ///< Maximum of articulated parts * @param mirrored Returns whether the part shall be flipped. * @return engine to add or INVALID_ENGINE */ -static EngineID GetNextArticPart(uint index, EngineID front_type, Vehicle *front = NULL, bool *mirrored = NULL) +static EngineID GetNextArticulatedPart(uint index, EngineID front_type, Vehicle *front = NULL, bool *mirrored = NULL) { assert(front == NULL || front->engine_type == front_type); @@ -55,7 +55,7 @@ uint CountArticulatedParts(EngineID engine_type, bool purchase_window) uint i; for (i = 1; i < MAX_ARTICULATED_PARTS; i++) { - if (GetNextArticPart(i, engine_type, v) == INVALID_ENGINE) break; + if (GetNextArticulatedPart(i, engine_type, v) == INVALID_ENGINE) break; } delete v; @@ -113,7 +113,7 @@ CargoArray GetCapacityOfArticulatedParts(EngineID engine) if (!HasBit(e->info.callback_mask, CBM_VEHICLE_ARTIC_ENGINE)) return capacity; for (uint i = 1; i < MAX_ARTICULATED_PARTS; i++) { - EngineID artic_engine = GetNextArticPart(i, engine); + EngineID artic_engine = GetNextArticulatedPart(i, engine); if (artic_engine == INVALID_ENGINE) break; cargo_capacity = GetVehicleDefaultCapacity(artic_engine, &cargo_type); @@ -138,7 +138,7 @@ bool IsArticulatedVehicleRefittable(EngineID engine) if (!HasBit(e->info.callback_mask, CBM_VEHICLE_ARTIC_ENGINE)) return false; for (uint i = 1; i < MAX_ARTICULATED_PARTS; i++) { - EngineID artic_engine = GetNextArticPart(i, engine); + EngineID artic_engine = GetNextArticulatedPart(i, engine); if (artic_engine == INVALID_ENGINE) break; if (IsEngineRefittable(artic_engine)) return true; @@ -165,7 +165,7 @@ void GetArticulatedRefitMasks(EngineID engine, bool include_initial_cargo_type, if (!HasBit(e->info.callback_mask, CBM_VEHICLE_ARTIC_ENGINE)) return; for (uint i = 1; i < MAX_ARTICULATED_PARTS; i++) { - EngineID artic_engine = GetNextArticPart(i, engine); + EngineID artic_engine = GetNextArticulatedPart(i, engine); if (artic_engine == INVALID_ENGINE) break; veh_cargos = GetAvailableVehicleCargoTypes(artic_engine, include_initial_cargo_type); @@ -223,7 +223,7 @@ bool IsArticulatedVehicleCarryingDifferentCargos(const Vehicle *v, CargoID *carg switch (v->type) { case VEH_TRAIN: - v = Train::From(v)->HasArticulatedPart() ? Train::From(v)->GetNextArticPart() : NULL; + v = Train::From(v)->HasArticulatedPart() ? Train::From(v)->GetNextArticulatedPart() : NULL; break; case VEH_ROAD: @@ -270,7 +270,7 @@ void CheckConsistencyOfArticulatedVehicle(const Vehicle *v) switch (v->type) { case VEH_TRAIN: - v = Train::From(v)->HasArticulatedPart() ? Train::From(v)->GetNextArticPart() : NULL; + v = Train::From(v)->HasArticulatedPart() ? Train::From(v)->GetNextArticulatedPart() : NULL; break; case VEH_ROAD: @@ -306,7 +306,7 @@ void AddArticulatedParts(Vehicle *first) Vehicle *v = first; for (uint i = 1; i < MAX_ARTICULATED_PARTS; i++) { bool flip_image; - EngineID engine_type = GetNextArticPart(i, first->engine_type, first, &flip_image); + EngineID engine_type = GetNextArticulatedPart(i, first->engine_type, first, &flip_image); if (engine_type == INVALID_ENGINE) return; /* In the (very rare) case the GRF reported wrong number of articulated parts diff --git a/src/train.h b/src/train.h index 1b472d4c4e..fcdadc4817 100644 --- a/src/train.h +++ b/src/train.h @@ -267,7 +267,7 @@ struct Train : public GroundVehicle { * Result is undefined for normal engine. * @return next part of articulated engine */ - FORCEINLINE Train *GetNextArticPart() const + FORCEINLINE Train *GetNextArticulatedPart() const { assert(this->HasArticulatedPart()); return this->Next(); @@ -302,7 +302,7 @@ struct Train : public GroundVehicle { FORCEINLINE Train *GetLastEnginePart() { Train *v = this; - while (v->HasArticulatedPart()) v = v->GetNextArticPart(); + while (v->HasArticulatedPart()) v = v->GetNextArticulatedPart(); return v; } @@ -313,7 +313,7 @@ struct Train : public GroundVehicle { FORCEINLINE Train *GetNextVehicle() const { const Train *v = this; - while (v->HasArticulatedPart()) v = v->GetNextArticPart(); + while (v->HasArticulatedPart()) v = v->GetNextArticulatedPart(); /* v now contains the last artic part in the engine */ return v->Next(); diff --git a/src/vehicle.cpp b/src/vehicle.cpp index 397c13a99a..2222c0bc9c 100644 --- a/src/vehicle.cpp +++ b/src/vehicle.cpp @@ -2373,7 +2373,7 @@ void GetVehicleSet(VehicleSet &set, Vehicle *v, uint8 num_vehicles) u = u->GetFirstEnginePart(); while (u->index != v->index) { set.Include(u->index); - u = u->GetNextArticPart(); + u = u->GetNextArticulatedPart(); } } diff --git a/src/vehicle_cmd.cpp b/src/vehicle_cmd.cpp index a3fa060483..b0766d349b 100644 --- a/src/vehicle_cmd.cpp +++ b/src/vehicle_cmd.cpp @@ -764,7 +764,7 @@ CommandCost CmdCloneVehicle(TileIndex tile, DoCommandFlag flags, uint32 p1, uint } if (w->type == VEH_TRAIN && Train::From(w)->HasArticulatedPart()) { - w = Train::From(w)->GetNextArticPart(); + w = Train::From(w)->GetNextArticulatedPart(); } else if (w->type == VEH_ROAD && RoadVehicle::From(w)->HasArticulatedPart()) { w = w->Next(); } else { @@ -780,7 +780,7 @@ CommandCost CmdCloneVehicle(TileIndex tile, DoCommandFlag flags, uint32 p1, uint } if (v->type == VEH_TRAIN && Train::From(v)->HasArticulatedPart()) { - v = Train::From(v)->GetNextArticPart(); + v = Train::From(v)->GetNextArticulatedPart(); } else if (v->type == VEH_ROAD && RoadVehicle::From(v)->HasArticulatedPart()) { v = v->Next(); } else {