--- a/aircraft_cmd.c Sat Oct 29 20:58:26 2005 +0000
+++ b/aircraft_cmd.c Sat Oct 29 21:54:28 2005 +0000
@@ -251,6 +251,7 @@
v->max_age = e->lifelength * 366;
_new_aircraft_id = v->index;
+ _new_vehicle_id = v->index;
v->u.air.pos = MAX_ELEMENTS;
--- a/roadveh_cmd.c Sat Oct 29 20:58:26 2005 +0000
+++ b/roadveh_cmd.c Sat Oct 29 21:54:28 2005 +0000
@@ -176,6 +176,7 @@
v->reliability_spd_dec = e->reliability_spd_dec;
v->max_age = e->lifelength * 366;
_new_roadveh_id = v->index;
+ _new_vehicle_id = v->index;
v->string_id = STR_SV_ROADVEH_NAME;
--- a/ship_cmd.c Sat Oct 29 20:58:26 2005 +0000
+++ b/ship_cmd.c Sat Oct 29 21:54:28 2005 +0000
@@ -907,6 +907,7 @@
v->reliability_spd_dec = e->reliability_spd_dec;
v->max_age = e->lifelength * 366;
_new_ship_id = v->index;
+ _new_vehicle_id = v->index;
v->string_id = STR_SV_SHIP_NAME;
v->u.ship.state = 0x80;
--- a/train_cmd.c Sat Oct 29 20:58:26 2005 +0000
+++ b/train_cmd.c Sat Oct 29 21:54:28 2005 +0000
@@ -471,6 +471,7 @@
v->cur_image = 0xAC2;
_new_wagon_id = v->index;
+ _new_vehicle_id = v->index;
VehiclePositionChanged(v);
TrainConsistChanged(GetFirstVehicleInChain(v));
@@ -632,6 +633,7 @@
v->string_id = STR_SV_TRAIN_NAME;
v->u.rail.railtype = e->railtype;
_new_train_id = v->index;
+ _new_vehicle_id = v->index;
v->service_interval = _patches.servint_trains;
v->date_of_last_service = _date;
--- a/vehicle.c Sat Oct 29 20:58:26 2005 +0000
+++ b/vehicle.c Sat Oct 29 21:54:28 2005 +0000
@@ -1403,13 +1403,6 @@
}
}
-static VehicleID * _new_vehicle_id_proc_table[] = {
- &_new_train_id,
- &_new_roadveh_id,
- &_new_ship_id,
- &_new_aircraft_id,
-};
-
/** Clone a vehicle. If it is a train, it will clone all the cars too
* @param x,y depot where the cloned vehicle is build
* @param p1 the original vehicle's index
@@ -1420,7 +1413,7 @@
Vehicle *v_front, *v;
Vehicle *w_front, *w, *w_rear;
int cost, total_cost = 0;
- VehicleID *new_id;
+// VehicleID *new_id;
if (!IsVehicleIndex(p1))
return CMD_ERROR;
@@ -1444,8 +1437,6 @@
if (v->type == VEH_Train && v->subtype != TS_Front_Engine) return CMD_ERROR;
- new_id = _new_vehicle_id_proc_table[v->type - VEH_Train];
-
do {
cost = DoCommand(x, y, v->engine_type, 3, flags, CMD_BUILD_VEH(v->type));
@@ -1454,11 +1445,7 @@
total_cost += cost;
if (flags & DC_EXEC) {
- if (v->type == VEH_Train && RailVehInfo(v->engine_type)->flags & RVI_WAGON) {
- w = GetVehicle(_new_wagon_id);
- } else {
- w = GetVehicle(*new_id);
- }
+ w= GetVehicle(_new_vehicle_id);
if (v->type != VEH_Road) { // road vehicles can't be refitted
if (v->cargo_type != w->cargo_type) {
@@ -1468,7 +1455,6 @@
if (v->type == VEH_Train && v->subtype != TS_Front_Engine) {
// this s a train car
-
// add this unit to the end of the train
DoCommand(x, y, (w_rear->index << 16) | w->index, 1, flags, CMD_MOVE_RAIL_VEHICLE);
} else {
@@ -1521,8 +1507,8 @@
if (CmdFailed(cost)) return cost;
if (flags & DC_EXEC) {
- new_v = GetVehicle(*_new_vehicle_id_proc_table[old_v->type - VEH_Train]);
- *w = new_v;
+ new_v = GetVehicle(_new_vehicle_id);
+ *w = new_v; //we changed the vehicle, so MaybeReplaceVehicle needs to work on the new one. Now we tell it what the new one is
/* refit if needed */
if (new_v->type != VEH_Road) { // road vehicles can't be refitted
--- a/vehicle.h Sat Oct 29 20:58:26 2005 +0000
+++ b/vehicle.h Sat Oct 29 21:54:28 2005 +0000
@@ -442,6 +442,7 @@
VARDEF VehicleID _new_aircraft_id;
VARDEF VehicleID _new_ship_id;
VARDEF VehicleID _new_roadveh_id;
+VARDEF VehicleID _new_vehicle_id;
VARDEF uint16 _aircraft_refit_capacity;
VARDEF byte _cmd_build_rail_veh_score;