diff --git a/src/saveload/extended_ver_sl.cpp b/src/saveload/extended_ver_sl.cpp index 4e28088d65..727a0350e6 100644 --- a/src/saveload/extended_ver_sl.cpp +++ b/src/saveload/extended_ver_sl.cpp @@ -55,7 +55,7 @@ const SlxiSubChunkInfo _sl_xv_sub_chunk_infos[] = { { XSLFI_TIMETABLES_START_TICKS, XSCF_NULL, WALLCLOCK_NETWORK_COMPATIBLE ? 0 : 1, 1, "timetable_start_ticks", NULL, NULL, NULL }, { XSLFI_TOWN_CARGO_ADJ, XSCF_IGNORABLE_UNKNOWN, 1, 1, "town_cargo_adj", NULL, NULL, NULL }, { XSLFI_SIG_TUNNEL_BRIDGE, XSCF_NULL, 1, 1, "signal_tunnel_bridge", NULL, NULL, NULL }, - { XSLFI_IMPROVED_BREAKDOWNS, XSCF_NULL, 1, 1, "improved_breakdowns", NULL, NULL, NULL }, + { XSLFI_IMPROVED_BREAKDOWNS, XSCF_NULL, 2, 2, "improved_breakdowns", NULL, NULL, NULL }, { XSLFI_TT_WAIT_IN_DEPOT, XSCF_NULL, 1, 1, "tt_wait_in_depot", NULL, NULL, NULL }, { XSLFI_AUTO_TIMETABLE, XSCF_NULL, 3, 3, "auto_timetables", NULL, NULL, NULL }, { XSLFI_VEHICLE_REPAIR_COST, XSCF_NULL, 1, 1, "vehicle_repair_cost", NULL, NULL, NULL }, diff --git a/src/saveload/vehicle_sl.cpp b/src/saveload/vehicle_sl.cpp index 35f6418392..c7078999fc 100644 --- a/src/saveload/vehicle_sl.cpp +++ b/src/saveload/vehicle_sl.cpp @@ -760,7 +760,8 @@ const SaveLoad *GetVehicleDescription(VehicleType vt) SLE_CONDNULL(2, 2, 19), SLE_CONDVAR(Train, gv_flags, SLE_UINT16, 139, SL_MAX_VERSION), SLE_CONDNULL(11, 2, 143), // old reserved space - SLE_CONDVAR_X(Train, reverse_distance, SLE_UINT16, 0, SL_MAX_VERSION, SlXvFeatureTest(XSLFTO_AND, XSLFI_REVERSE_AT_WAYPOINT)), + SLE_CONDVAR_X(Train, reverse_distance, SLE_UINT16, 0, SL_MAX_VERSION, SlXvFeatureTest(XSLFTO_AND, XSLFI_REVERSE_AT_WAYPOINT)), + SLE_CONDVAR_X(Train, critical_breakdown_count, SLE_UINT8, 0, SL_MAX_VERSION, SlXvFeatureTest(XSLFTO_AND, XSLFI_IMPROVED_BREAKDOWNS, 2)), SLE_END() }; diff --git a/src/settings_type.h b/src/settings_type.h index 5a5e2c2a8a..f703e95da4 100644 --- a/src/settings_type.h +++ b/src/settings_type.h @@ -509,7 +509,7 @@ struct VehicleSettings { byte road_side; ///< the side of the road vehicles drive on uint8 plane_crashes; ///< number of plane crashes, 0 = none, 1 = reduced, 2 = normal bool adjacent_crossings; ///< enable closing of adjacent level crossings - bool improved_breakdowns; ///< different types, chances and serverities of breakdowns + bool improved_breakdowns; ///< different types, chances and severities of breakdowns bool pay_for_repair; ///< pay for repairing vehicle uint8 repair_cost; ///< cost of repairing vehicle }; diff --git a/src/train.h b/src/train.h index 50500e21d5..2e59ee980e 100644 --- a/src/train.h +++ b/src/train.h @@ -108,6 +108,7 @@ struct Train FINAL : public GroundVehicle { TrackBitsByte track; TrainForceProceedingByte force_proceed; RailTypeByte railtype; + byte critical_breakdown_count; ///< Counter for the number of critical breakdowns since last service RailTypes compatible_railtypes; /** Ticks waiting in front of a signal, ticks being stuck or a counter for forced proceeding through signals. */ diff --git a/src/train_cmd.cpp b/src/train_cmd.cpp index 2fd0b22f5f..c74b2ad143 100644 --- a/src/train_cmd.cpp +++ b/src/train_cmd.cpp @@ -192,6 +192,12 @@ void Train::ConsistChanged(ConsistChangeFlags allowed_changes) u->tcache.user_def_data = GetVehicleProperty(u, PROP_TRAIN_USER_DATA, u->tcache.user_def_data); this->InvalidateNewGRFCache(); u->InvalidateNewGRFCache(); + + if (!u->IsArticulatedPart()) { + if (u->IsEngine() || u->IsMultiheaded()) { + this->tcache.cached_num_engines++; + } + } } for (Train *u = this; u != NULL; u = u->Next()) { @@ -234,13 +240,13 @@ void Train::ConsistChanged(ConsistChangeFlags allowed_changes) /* max speed is the minimum of the speed limits of all vehicles in the consist */ if ((rvi_u->railveh_type != RAILVEH_WAGON || _settings_game.vehicle.wagon_speed_limits) && !UsesWagonOverride(u)) { uint16 speed = GetVehicleProperty(u, PROP_TRAIN_SPEED, rvi_u->max_speed); - if (HasBit(u->flags, VRF_NEED_REPAIR)) speed = u->vcache.cached_max_speed; + if (HasBit(u->flags, VRF_NEED_REPAIR) && this->IsFrontEngine()) { + for (uint i = 0; i < u->critical_breakdown_count; i++) { + speed = min(speed - (speed / (this->tcache.cached_num_engines + 2)) + 1, speed); + } + } if (speed != 0) max_speed = min(speed, max_speed); } - - if (u->IsEngine() || u-> IsMultiheaded()) { - this->tcache.cached_num_engines++; - } } uint16 new_cap = e_u->DetermineCapacity(u); diff --git a/src/vehicle.cpp b/src/vehicle.cpp index 808d6ca133..844e536d27 100644 --- a/src/vehicle.cpp +++ b/src/vehicle.cpp @@ -106,6 +106,7 @@ void VehicleServiceInDepot(Vehicle *v) if (v->Next() != NULL) VehicleServiceInDepot(v->Next()); if (!(Train::From(v)->IsEngine()) && !(Train::From(v)->IsRearDualheaded())) return; ClrBit(Train::From(v)->flags,VRF_NEED_REPAIR); + Train::From(v)->critical_breakdown_count = 0; const RailVehicleInfo *rvi = &e->u.rail; v->vcache.cached_max_speed = rvi->max_speed; if (Train::From(v)->IsFrontEngine()) { @@ -1469,17 +1470,12 @@ bool Vehicle::HandleBreakdown() } /* Max Speed reduction*/ if (_settings_game.vehicle.improved_breakdowns) { - const Engine *e = Engine::Get(this->engine_type); - const RailVehicleInfo *rvi = &e->u.rail; - if (!HasBit(Train::From(this)->flags,VRF_NEED_REPAIR)) { - if (rvi->max_speed > this->vcache.cached_max_speed) { - this->vcache.cached_max_speed = rvi->max_speed; - } + if (!HasBit(Train::From(this)->flags, VRF_NEED_REPAIR)) { + SetBit(Train::From(this)->flags, VRF_NEED_REPAIR); + Train::From(this)->critical_breakdown_count = 1; + } else if (Train::From(this)->critical_breakdown_count != 255) { + Train::From(this)->critical_breakdown_count++; } - uint16 target_max_speed = min(this->vcache.cached_max_speed - - (this->vcache.cached_max_speed >> 1) / Train::From(this->First())->tcache.cached_num_engines + 1, this->vcache.cached_max_speed); - this->vcache.cached_max_speed = max(target_max_speed, min(rvi->max_speed / 4, 28)); - SetBit(Train::From(this)->flags, VRF_NEED_REPAIR); Train::From(this->First())->ConsistChanged(CCF_TRACK); } /* FALL THROUGH */