diff --git a/src/aircraft.h b/src/aircraft.h --- a/src/aircraft.h +++ b/src/aircraft.h @@ -128,7 +128,7 @@ struct Aircraft : public Vehicle { Aircraft() { this->type = VEH_AIRCRAFT; } /** We want to 'destruct' the right class. */ - virtual ~Aircraft() {} + virtual ~Aircraft() { this->PreDestructor(); } const char *GetTypeString() const { return "aircraft"; } void MarkDirty(); diff --git a/src/roadveh.h b/src/roadveh.h --- a/src/roadveh.h +++ b/src/roadveh.h @@ -72,7 +72,7 @@ struct RoadVehicle : public Vehicle { RoadVehicle() { this->type = VEH_ROAD; } /** We want to 'destruct' the right class. */ - virtual ~RoadVehicle() {} + virtual ~RoadVehicle() { this->PreDestructor(); } const char *GetTypeString() const { return "road vehicle"; } void MarkDirty(); diff --git a/src/ship.h b/src/ship.h --- a/src/ship.h +++ b/src/ship.h @@ -37,7 +37,7 @@ struct Ship: public Vehicle { Ship() { this->type = VEH_SHIP; } /** We want to 'destruct' the right class. */ - virtual ~Ship() {} + virtual ~Ship() { this->PreDestructor(); } const char *GetTypeString() const { return "ship"; } void MarkDirty(); diff --git a/src/train.h b/src/train.h --- a/src/train.h +++ b/src/train.h @@ -262,7 +262,7 @@ struct Train : public Vehicle { Train() { this->type = VEH_TRAIN; } /** We want to 'destruct' the right class. */ - virtual ~Train() {} + virtual ~Train() { this->PreDestructor(); } const char *GetTypeString() const { return "train"; } void MarkDirty(); diff --git a/src/vehicle.cpp b/src/vehicle.cpp --- a/src/vehicle.cpp +++ b/src/vehicle.cpp @@ -562,7 +562,7 @@ bool IsEngineCountable(const Vehicle *v) } } -Vehicle::~Vehicle() +void Vehicle::PreDestructor() { if (IsValidStationID(this->last_station_visited)) { GetStation(this->last_station_visited)->loading_vehicles.remove(this); @@ -579,8 +579,6 @@ Vehicle::~Vehicle() if (this->IsPrimaryVehicle()) DecreaseGroupNumVehicle(this->group_id); } - DeleteVehicleNews(this->index, INVALID_STRING_ID); - this->QuickFree(); if (this->type == VEH_ROAD) ClearSlot(this); @@ -589,10 +587,7 @@ Vehicle::~Vehicle() } this->cargo.Truncate(0); - UpdateVehiclePosHash(this, INVALID_COORD, 0); - this->next_hash = NULL; - this->next_new_hash = NULL; - if (IsPlayerBuildableVehicleType(this)) DeleteVehicleOrders(this); + DeleteVehicleOrders(this); /* Now remove any artic part. This will trigger an other * destroy vehicle, which on his turn can remove any @@ -600,6 +595,15 @@ Vehicle::~Vehicle() if ((this->type == VEH_TRAIN && EngineHasArticPart(this)) || (this->type == VEH_ROAD && RoadVehHasArticPart(this))) { delete this->next; } +} + +Vehicle::~Vehicle() +{ + UpdateVehiclePosHash(this, INVALID_COORD, 0); + this->next_hash = NULL; + this->next_new_hash = NULL; + + DeleteVehicleNews(this->index, INVALID_STRING_ID); new (this) InvalidVehicle(); } diff --git a/src/vehicle.h b/src/vehicle.h --- a/src/vehicle.h +++ b/src/vehicle.h @@ -347,6 +347,8 @@ struct Vehicle : PoolItem