mirror
/
OpenTTD
mirror of https://github.com/OpenTTD/OpenTTD
1
0
Fork 0
Code Issues

(svn r12588) -Codechange: do not access the destination of an order directly.

release/0.7
rubidium 2008-04-06 07:48:51 +00:00
parent 8cd1795fe3
commit d6623cf654
18 changed files with 116 additions and 108 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/ai/default/default.cpp
View File

@ -263,8 +263,8 @@ static EngineID AiChooseAircraftToReplaceWith(const Player* p, const Vehicle* v)
FOR_VEHICLE_ORDERS(v, o) { FOR_VEHICLE_ORDERS(v, o) {
if (!o->IsValid()) continue; if (!o->IsValid()) continue;
if (!IsValidStationID(o->dest)) continue; if (!IsValidStationID(o->GetDestination())) continue;
const Station *st = GetStation(o->dest); const Station *st = GetStation(o->GetDestination());
if (!(st->facilities & FACIL_AIRPORT)) continue; if (!(st->facilities & FACIL_AIRPORT)) continue;
AirportFTAClass::Flags flags = st->Airport()->flags; AirportFTAClass::Flags flags = st->Airport()->flags;
@ -3662,7 +3662,7 @@ static void AiStateRemoveStation(Player *p)
byte *in_use = MallocT<byte>(GetMaxStationIndex() + 1); byte *in_use = MallocT<byte>(GetMaxStationIndex() + 1);
memset(in_use, 0, GetMaxStationIndex() + 1); memset(in_use, 0, GetMaxStationIndex() + 1);
FOR_ALL_ORDERS(ord) { FOR_ALL_ORDERS(ord) {
if (ord->IsType(OT_GOTO_STATION)) in_use[ord->dest] = 1; if (ord->IsType(OT_GOTO_STATION)) in_use[ord->GetDestination()] = 1;
} }
// Go through all stations and delete those that aren't in use // Go through all stations and delete those that aren't in use

2
src/ai/trolly/trolly.cpp
View File

@ -553,7 +553,7 @@ static bool AiNew_CheckVehicleStation(Player *p, Station *st)
const Order *order; const Order *order;
FOR_VEHICLE_ORDERS(v, order) { FOR_VEHICLE_ORDERS(v, order) {
if (order->IsType(OT_GOTO_STATION) && GetStation(order->dest) == st) { if (order->IsType(OT_GOTO_STATION) && GetStation(order->GetDestination()) == st) {
// This vehicle has this city in its list // This vehicle has this city in its list
count++; count++;
} }

10
src/aircraft_cmd.cpp
View File

@ -705,7 +705,7 @@ static void CheckIfAircraftNeedsService(Vehicle *v)
return; return;
} }
const Station *st = GetStation(v->current_order.dest); const Station *st = GetStation(v->current_order.GetDestination());
/* only goto depot if the target airport has terminals (eg. it is airport) */ /* only goto depot if the target airport has terminals (eg. it is airport) */
if (st->IsValid() && st->airport_tile != 0 && st->Airport()->terminals != NULL) { if (st->IsValid() && st->airport_tile != 0 && st->Airport()->terminals != NULL) {
// printf("targetairport = %d, st->index = %d\n", v->u.air.targetairport, st->index); // printf("targetairport = %d, st->index = %d\n", v->u.air.targetairport, st->index);
@ -1054,7 +1054,7 @@ static bool AircraftController(Vehicle *v)
/* Jump into our "holding pattern" state machine if possible */ /* Jump into our "holding pattern" state machine if possible */
if (v->u.air.pos >= afc->nofelements) { if (v->u.air.pos >= afc->nofelements) {
v->u.air.pos = v->u.air.previous_pos = AircraftGetEntryPoint(v, afc); v->u.air.pos = v->u.air.previous_pos = AircraftGetEntryPoint(v, afc);
} else if (v->u.air.targetairport != v->current_order.dest) { } else if (v->u.air.targetairport != v->current_order.GetDestination()) {
/* If not possible, just get out of here fast */ /* If not possible, just get out of here fast */
v->u.air.state = FLYING; v->u.air.state = FLYING;
UpdateAircraftCache(v); UpdateAircraftCache(v);
@ -1501,7 +1501,7 @@ static void AircraftNextAirportPos_and_Order(Vehicle *v)
{ {
if (v->current_order.IsType(OT_GOTO_STATION) || if (v->current_order.IsType(OT_GOTO_STATION) ||
v->current_order.IsType(OT_GOTO_DEPOT)) v->current_order.IsType(OT_GOTO_DEPOT))
v->u.air.targetairport = v->current_order.dest; v->u.air.targetairport = v->current_order.GetDestination();
const AirportFTAClass *apc = GetStation(v->u.air.targetairport)->Airport(); const AirportFTAClass *apc = GetStation(v->u.air.targetairport)->Airport();
v->u.air.pos = v->u.air.previous_pos = AircraftGetEntryPoint(v, apc); v->u.air.pos = v->u.air.previous_pos = AircraftGetEntryPoint(v, apc);
@ -1610,7 +1610,7 @@ static void AircraftEventHandler_InHangar(Vehicle *v, const AirportFTAClass *apc
if (AirportHasBlock(v, &apc->layout[v->u.air.pos], apc)) return; if (AirportHasBlock(v, &apc->layout[v->u.air.pos], apc)) return;
/* We are already at the target airport, we need to find a terminal */ /* We are already at the target airport, we need to find a terminal */
if (v->current_order.dest == v->u.air.targetairport) { if (v->current_order.GetDestination() == v->u.air.targetairport) {
/* FindFreeTerminal: /* FindFreeTerminal:
* 1. Find a free terminal, 2. Occupy it, 3. Set the vehicle's state to that terminal */ * 1. Find a free terminal, 2. Occupy it, 3. Set the vehicle's state to that terminal */
if (v->subtype == AIR_HELICOPTER) { if (v->subtype == AIR_HELICOPTER) {
@ -1660,7 +1660,7 @@ static void AircraftEventHandler_AtTerminal(Vehicle *v, const AirportFTAClass *a
v->u.air.state = (v->subtype == AIR_HELICOPTER) ? HELITAKEOFF : TAKEOFF; v->u.air.state = (v->subtype == AIR_HELICOPTER) ? HELITAKEOFF : TAKEOFF;
break; break;
case OT_GOTO_DEPOT: // visit hangar for serivicing, sale, etc. case OT_GOTO_DEPOT: // visit hangar for serivicing, sale, etc.
if (v->current_order.dest == v->u.air.targetairport) { if (v->current_order.GetDestination() == v->u.air.targetairport) {
v->u.air.state = HANGAR; v->u.air.state = HANGAR;
} else { } else {
v->u.air.state = (v->subtype == AIR_HELICOPTER) ? HELITAKEOFF : TAKEOFF; v->u.air.state = (v->subtype == AIR_HELICOPTER) ? HELITAKEOFF : TAKEOFF;

58
src/disaster_cmd.cpp
View File

@ -206,29 +206,29 @@ static void DisasterTick_Zeppeliner(Vehicle *v)
v->tick_counter++; v->tick_counter++;
if (v->current_order.dest < 2) { if (v->current_order.GetDestination() < 2) {
if (HasBit(v->tick_counter, 0)) return; if (HasBit(v->tick_counter, 0)) return;
GetNewVehiclePosResult gp = GetNewVehiclePos(v); GetNewVehiclePosResult gp = GetNewVehiclePos(v);
SetDisasterVehiclePos(v, gp.x, gp.y, v->z_pos); SetDisasterVehiclePos(v, gp.x, gp.y, v->z_pos);
if (v->current_order.dest == 1) { if (v->current_order.GetDestination() == 1) {
if (++v->age == 38) { if (++v->age == 38) {
v->current_order.dest = 2; v->current_order.SetDestination(2);
v->age = 0; v->age = 0;
} }
if (GB(v->tick_counter, 0, 3) == 0) CreateEffectVehicleRel(v, 0, -17, 2, EV_SMOKE); if (GB(v->tick_counter, 0, 3) == 0) CreateEffectVehicleRel(v, 0, -17, 2, EV_SMOKE);
} else if (v->current_order.dest == 0) { } else if (v->current_order.GetDestination() == 0) {
tile = v->tile; tile = v->tile;
if (IsValidTile(tile) && if (IsValidTile(tile) &&
IsTileType(tile, MP_STATION) && IsTileType(tile, MP_STATION) &&
IsAirport(tile) && IsAirport(tile) &&
IsHumanPlayer(GetTileOwner(tile))) { IsHumanPlayer(GetTileOwner(tile))) {
v->current_order.dest = 1; v->current_order.SetDestination(1);
v->age = 0; v->age = 0;
SetDParam(0, GetStationIndex(tile)); SetDParam(0, GetStationIndex(tile));
@ -243,7 +243,7 @@ static void DisasterTick_Zeppeliner(Vehicle *v)
return; return;
} }
if (v->current_order.dest > 2) { if (v->current_order.GetDestination() > 2) {
if (++v->age <= 13320) return; if (++v->age <= 13320) return;
tile = v->tile; tile = v->tile;
@ -284,7 +284,7 @@ static void DisasterTick_Zeppeliner(Vehicle *v)
EV_EXPLOSION_SMALL); EV_EXPLOSION_SMALL);
} }
} else if (v->age == 350) { } else if (v->age == 350) {
v->current_order.dest = 3; v->current_order.SetDestination(3);
v->age = 0; v->age = 0;
} }
@ -312,7 +312,7 @@ static void DisasterTick_Ufo(Vehicle *v)
v->u.disaster.image_override = (HasBit(++v->tick_counter, 3)) ? SPR_UFO_SMALL_SCOUT_DARKER : SPR_UFO_SMALL_SCOUT; v->u.disaster.image_override = (HasBit(++v->tick_counter, 3)) ? SPR_UFO_SMALL_SCOUT_DARKER : SPR_UFO_SMALL_SCOUT;
if (v->current_order.dest == 0) { if (v->current_order.GetDestination() == 0) {
/* Fly around randomly */ /* Fly around randomly */
int x = TileX(v->dest_tile) * TILE_SIZE; int x = TileX(v->dest_tile) * TILE_SIZE;
int y = TileY(v->dest_tile) * TILE_SIZE; int y = TileY(v->dest_tile) * TILE_SIZE;
@ -326,7 +326,7 @@ static void DisasterTick_Ufo(Vehicle *v)
v->dest_tile = RandomTile(); v->dest_tile = RandomTile();
return; return;
} }
v->current_order.dest = 1; v->current_order.SetDestination(1);
FOR_ALL_VEHICLES(u) { FOR_ALL_VEHICLES(u) {
if (u->type == VEH_ROAD && IsHumanPlayer(u->owner)) { if (u->type == VEH_ROAD && IsHumanPlayer(u->owner)) {
@ -405,7 +405,7 @@ static void DisasterTick_Airplane(Vehicle *v)
{ {
v->tick_counter++; v->tick_counter++;
v->u.disaster.image_override = v->u.disaster.image_override =
(v->current_order.dest == 1 && HasBit(v->tick_counter, 2)) ? SPR_F_15_FIRING : 0; (v->current_order.GetDestination() == 1 && HasBit(v->tick_counter, 2)) ? SPR_F_15_FIRING : 0;
GetNewVehiclePosResult gp = GetNewVehiclePos(v); GetNewVehiclePosResult gp = GetNewVehiclePos(v);
SetDisasterVehiclePos(v, gp.x, gp.y, v->z_pos); SetDisasterVehiclePos(v, gp.x, gp.y, v->z_pos);
@ -415,7 +415,7 @@ static void DisasterTick_Airplane(Vehicle *v)
return; return;
} }
if (v->current_order.dest == 2) { if (v->current_order.GetDestination() == 2) {
if (GB(v->tick_counter, 0, 2) == 0) { if (GB(v->tick_counter, 0, 2) == 0) {
Industry *i = GetIndustry(v->dest_tile); Industry *i = GetIndustry(v->dest_tile);
int x = TileX(i->xy) * TILE_SIZE; int x = TileX(i->xy) * TILE_SIZE;
@ -428,13 +428,13 @@ static void DisasterTick_Airplane(Vehicle *v)
GB(r, 12, 4), GB(r, 12, 4),
EV_EXPLOSION_SMALL); EV_EXPLOSION_SMALL);
if (++v->age >= 55) v->current_order.dest = 3; if (++v->age >= 55) v->current_order.SetDestination(3);
} }
} else if (v->current_order.dest == 1) { } else if (v->current_order.GetDestination() == 1) {
if (++v->age == 112) { if (++v->age == 112) {
Industry *i; Industry *i;
v->current_order.dest = 2; v->current_order.SetDestination(2);
v->age = 0; v->age = 0;
i = GetIndustry(v->dest_tile); i = GetIndustry(v->dest_tile);
@ -444,7 +444,7 @@ static void DisasterTick_Airplane(Vehicle *v)
AddNewsItem(STR_B002_OIL_REFINERY_EXPLOSION, NM_THIN, NF_VIEWPORT | NF_TILE, NT_ACCIDENT, DNC_NONE, i->xy, 0); AddNewsItem(STR_B002_OIL_REFINERY_EXPLOSION, NM_THIN, NF_VIEWPORT | NF_TILE, NT_ACCIDENT, DNC_NONE, i->xy, 0);
SndPlayTileFx(SND_12_EXPLOSION, i->xy); SndPlayTileFx(SND_12_EXPLOSION, i->xy);
} }
} else if (v->current_order.dest == 0) { } else if (v->current_order.GetDestination() == 0) {
int x, y; int x, y;
TileIndex tile; TileIndex tile;
uint ind; uint ind;
@ -461,7 +461,7 @@ static void DisasterTick_Airplane(Vehicle *v)
v->dest_tile = ind; v->dest_tile = ind;
if (GetIndustrySpec(GetIndustry(ind)->type)->behaviour & INDUSTRYBEH_AIRPLANE_ATTACKS) { if (GetIndustrySpec(GetIndustry(ind)->type)->behaviour & INDUSTRYBEH_AIRPLANE_ATTACKS) {
v->current_order.dest = 1; v->current_order.SetDestination(1);
v->age = 0; v->age = 0;
} }
} }
@ -478,7 +478,7 @@ static void DisasterTick_Helicopter(Vehicle *v)
{ {
v->tick_counter++; v->tick_counter++;
v->u.disaster.image_override = v->u.disaster.image_override =
(v->current_order.dest == 1 && HasBit(v->tick_counter, 2)) ? SPR_AH_64A_FIRING : 0; (v->current_order.GetDestination() == 1 && HasBit(v->tick_counter, 2)) ? SPR_AH_64A_FIRING : 0;
GetNewVehiclePosResult gp = GetNewVehiclePos(v); GetNewVehiclePosResult gp = GetNewVehiclePos(v);
SetDisasterVehiclePos(v, gp.x, gp.y, v->z_pos); SetDisasterVehiclePos(v, gp.x, gp.y, v->z_pos);
@ -488,7 +488,7 @@ static void DisasterTick_Helicopter(Vehicle *v)
return; return;
} }
if (v->current_order.dest == 2) { if (v->current_order.GetDestination() == 2) {
if (GB(v->tick_counter, 0, 2) == 0) { if (GB(v->tick_counter, 0, 2) == 0) {
Industry *i = GetIndustry(v->dest_tile); Industry *i = GetIndustry(v->dest_tile);
int x = TileX(i->xy) * TILE_SIZE; int x = TileX(i->xy) * TILE_SIZE;
@ -501,13 +501,13 @@ static void DisasterTick_Helicopter(Vehicle *v)
GB(r, 12, 4), GB(r, 12, 4),
EV_EXPLOSION_SMALL); EV_EXPLOSION_SMALL);
if (++v->age >= 55) v->current_order.dest = 3; if (++v->age >= 55) v->current_order.SetDestination(3);
} }
} else if (v->current_order.dest == 1) { } else if (v->current_order.GetDestination() == 1) {
if (++v->age == 112) { if (++v->age == 112) {
Industry *i; Industry *i;
v->current_order.dest = 2; v->current_order.SetDestination(2);
v->age = 0; v->age = 0;
i = GetIndustry(v->dest_tile); i = GetIndustry(v->dest_tile);
@ -517,7 +517,7 @@ static void DisasterTick_Helicopter(Vehicle *v)
AddNewsItem(STR_B003_FACTORY_DESTROYED_IN_SUSPICIOUS, NM_THIN, NF_VIEWPORT | NF_TILE, NT_ACCIDENT, DNC_NONE, i->xy, 0); AddNewsItem(STR_B003_FACTORY_DESTROYED_IN_SUSPICIOUS, NM_THIN, NF_VIEWPORT | NF_TILE, NT_ACCIDENT, DNC_NONE, i->xy, 0);
SndPlayTileFx(SND_12_EXPLOSION, i->xy); SndPlayTileFx(SND_12_EXPLOSION, i->xy);
} }
} else if (v->current_order.dest == 0) { } else if (v->current_order.GetDestination() == 0) {
int x, y; int x, y;
TileIndex tile; TileIndex tile;
uint ind; uint ind;
@ -534,7 +534,7 @@ static void DisasterTick_Helicopter(Vehicle *v)
v->dest_tile = ind; v->dest_tile = ind;
if (GetIndustrySpec(GetIndustry(ind)->type)->behaviour & INDUSTRYBEH_CHOPPER_ATTACKS) { if (GetIndustrySpec(GetIndustry(ind)->type)->behaviour & INDUSTRYBEH_CHOPPER_ATTACKS) {
v->current_order.dest = 1; v->current_order.SetDestination(1);
v->age = 0; v->age = 0;
} }
} }
@ -568,7 +568,7 @@ static void DisasterTick_Big_Ufo(Vehicle *v)
v->tick_counter++; v->tick_counter++;
if (v->current_order.dest == 1) { if (v->current_order.GetDestination() == 1) {
int x = TileX(v->dest_tile) * TILE_SIZE + TILE_SIZE / 2; int x = TileX(v->dest_tile) * TILE_SIZE + TILE_SIZE / 2;
int y = TileY(v->dest_tile) * TILE_SIZE + TILE_SIZE / 2; int y = TileY(v->dest_tile) * TILE_SIZE + TILE_SIZE / 2;
if (Delta(v->x_pos, x) + Delta(v->y_pos, y) >= 8) { if (Delta(v->x_pos, x) + Delta(v->y_pos, y) >= 8) {
@ -585,7 +585,7 @@ static void DisasterTick_Big_Ufo(Vehicle *v)
return; return;
} }
v->current_order.dest = 2; v->current_order.SetDestination(2);
FOR_ALL_VEHICLES(u) { FOR_ALL_VEHICLES(u) {
if (u->type == VEH_TRAIN || u->type == VEH_ROAD) { if (u->type == VEH_TRAIN || u->type == VEH_ROAD) {
@ -618,7 +618,7 @@ static void DisasterTick_Big_Ufo(Vehicle *v)
u->SetNext(w); u->SetNext(w);
InitializeDisasterVehicle(w, -6 * TILE_SIZE, v->y_pos, 0, DIR_SW, ST_Big_Ufo_Destroyer_Shadow); InitializeDisasterVehicle(w, -6 * TILE_SIZE, v->y_pos, 0, DIR_SW, ST_Big_Ufo_Destroyer_Shadow);
w->vehstatus |= VS_SHADOW; w->vehstatus |= VS_SHADOW;
} else if (v->current_order.dest == 0) { } else if (v->current_order.GetDestination() == 0) {
int x = TileX(v->dest_tile) * TILE_SIZE; int x = TileX(v->dest_tile) * TILE_SIZE;
int y = TileY(v->dest_tile) * TILE_SIZE; int y = TileY(v->dest_tile) * TILE_SIZE;
if (Delta(x, v->x_pos) + Delta(y, v->y_pos) >= TILE_SIZE) { if (Delta(x, v->x_pos) + Delta(y, v->y_pos) >= TILE_SIZE) {
@ -632,7 +632,7 @@ static void DisasterTick_Big_Ufo(Vehicle *v)
v->dest_tile = RandomTile(); v->dest_tile = RandomTile();
return; return;
} }
v->current_order.dest = 1; v->current_order.SetDestination(1);
tile_org = tile = RandomTile(); tile_org = tile = RandomTile();
do { do {
@ -669,10 +669,10 @@ static void DisasterTick_Big_Ufo_Destroyer(Vehicle *v)
return; return;
} }
if (v->current_order.dest == 0) { if (v->current_order.GetDestination() == 0) {
u = GetVehicle(v->u.disaster.big_ufo_destroyer_target); u = GetVehicle(v->u.disaster.big_ufo_destroyer_target);
if (Delta(v->x_pos, u->x_pos) > TILE_SIZE) return; if (Delta(v->x_pos, u->x_pos) > TILE_SIZE) return;
v->current_order.dest = 1; v->current_order.SetDestination(1);
CreateEffectVehicleRel(u, 0, 7, 8, EV_EXPLOSION_LARGE); CreateEffectVehicleRel(u, 0, 7, 8, EV_EXPLOSION_LARGE);
SndPlayVehicleFx(SND_12_EXPLOSION, u); SndPlayVehicleFx(SND_12_EXPLOSION, u);

2
src/industry_cmd.cpp
View File

@ -1969,7 +1969,7 @@ int WhoCanServiceIndustry(Industry* ind)
FOR_VEHICLE_ORDERS(v, o) { FOR_VEHICLE_ORDERS(v, o) {
if (o->IsType(OT_GOTO_STATION) && !HasBit(o->flags, OF_TRANSFER)) { if (o->IsType(OT_GOTO_STATION) && !HasBit(o->flags, OF_TRANSFER)) {
/* Vehicle visits a station to load or unload */ /* Vehicle visits a station to load or unload */
Station *st = GetStation(o->dest); Station *st = GetStation(o->GetDestination());
if (!st->IsValid()) continue; if (!st->IsValid()) continue;
/* Same cargo produced by industry is dropped here => not serviced by vehicle v */ /* Same cargo produced by industry is dropped here => not serviced by vehicle v */

4
src/npf.cpp
View File

@ -981,9 +981,9 @@ void NPFFillWithOrderData(NPFFindStationOrTileData* fstd, Vehicle* v)
* So only for train orders to stations we fill fstd->station_index, for all * So only for train orders to stations we fill fstd->station_index, for all
* others only dest_coords */ * others only dest_coords */
if (v->current_order.IsType(OT_GOTO_STATION) && v->type == VEH_TRAIN) { if (v->current_order.IsType(OT_GOTO_STATION) && v->type == VEH_TRAIN) {
fstd->station_index = v->current_order.dest; fstd->station_index = v->current_order.GetDestination();
/* Let's take the closest tile of the station as our target for trains */ /* Let's take the closest tile of the station as our target for trains */
fstd->dest_coords = CalcClosestStationTile(v->current_order.dest, v->tile); fstd->dest_coords = CalcClosestStationTile(fstd->station_index, v->tile);
} else { } else {
fstd->dest_coords = v->dest_tile; fstd->dest_coords = v->dest_tile;
fstd->station_index = INVALID_STATION; fstd->station_index = INVALID_STATION;

2
src/openttd.cpp
View File

@ -2305,7 +2305,7 @@ bool AfterLoadGame()
/* Update go to buoy orders because they are just waypoints */ /* Update go to buoy orders because they are just waypoints */
Order *order; Order *order;
FOR_ALL_ORDERS(order) { FOR_ALL_ORDERS(order) {
if (order->IsType(OT_GOTO_STATION) && GetStation(order->dest)->IsBuoy()) { if (order->IsType(OT_GOTO_STATION) && GetStation(order->GetDestination())->IsBuoy()) {
order->flags = 0; order->flags = 0;
} }
} }

16
src/order_base.h
View File

@ -28,6 +28,7 @@ private:
friend const struct SaveLoad *GetOrderDescription(); ///< Saving and loading of orders. friend const struct SaveLoad *GetOrderDescription(); ///< Saving and loading of orders.
OrderTypeByte type; ///< The type of order OrderTypeByte type; ///< The type of order
DestinationID dest; ///< The destination of the order.
CargoID refit_cargo; ///< Refit CargoID CargoID refit_cargo; ///< Refit CargoID
byte refit_subtype; ///< Refit subtype byte refit_subtype; ///< Refit subtype
@ -36,7 +37,6 @@ public:
Order *next; ///< Pointer to next order. If NULL, end of list Order *next; ///< Pointer to next order. If NULL, end of list
uint8 flags; uint8 flags;
DestinationID dest; ///< The destionation of the order.
uint16 wait_time; ///< How long in ticks to wait at the destination. uint16 wait_time; ///< How long in ticks to wait at the destination.
uint16 travel_time; ///< How long in ticks the journey to this destination should take. uint16 travel_time; ///< How long in ticks the journey to this destination should take.
@ -117,6 +117,20 @@ public:
*/ */
void FreeChain(); void FreeChain();
/**
* Gets the destination of this order.
* @pre IsType(OT_GOTO_WAYPOINT) || IsType(OT_GOTO_DEPOT) || IsType(OT_GOTO_STATION).
* @return the destination of the order.
*/
inline DestinationID GetDestination() const { return this->dest; }
/**
* Sets the destination of this order.
* @param destination the new destination of the order.
* @pre IsType(OT_GOTO_WAYPOINT) || IsType(OT_GOTO_DEPOT) || IsType(OT_GOTO_STATION).
*/
inline void SetDestination(DestinationID destination) { this->dest = destination; }
/** /**
* Is this order a refit order. * Is this order a refit order.
* @pre IsType(OT_GOTO_DEPOT) * @pre IsType(OT_GOTO_DEPOT)

48
src/order_cmd.cpp
View File

@ -160,7 +160,7 @@ Order UnpackOldOrder(uint16 packed)
* Sanity check * Sanity check
* TTD stores invalid orders as OT_NOTHING with non-zero flags/station * TTD stores invalid orders as OT_NOTHING with non-zero flags/station
*/ */
if (!order.IsValid() && (order.flags != 0 || order.dest != 0)) { if (!order.IsValid() && (order.flags != 0 || order.GetDestination() != 0)) {
order.MakeDummy(); order.MakeDummy();
} }
@ -234,8 +234,8 @@ static TileIndex GetOrderLocation(const Order& o)
{ {
switch (o.GetType()) { switch (o.GetType()) {
default: NOT_REACHED(); default: NOT_REACHED();
case OT_GOTO_STATION: return GetStation(o.dest)->xy; case OT_GOTO_STATION: return GetStation(o.GetDestination())->xy;
case OT_GOTO_DEPOT: return GetDepot(o.dest)->xy; case OT_GOTO_DEPOT: return GetDepot(o.GetDestination())->xy;
} }
} }
@ -267,10 +267,9 @@ CommandCost CmdInsertOrder(TileIndex tile, uint32 flags, uint32 p1, uint32 p2)
* and has the correct flags if any */ * and has the correct flags if any */
switch (new_order.GetType()) { switch (new_order.GetType()) {
case OT_GOTO_STATION: { case OT_GOTO_STATION: {
const Station *st; if (!IsValidStationID(new_order.GetDestination())) return CMD_ERROR;
if (!IsValidStationID(new_order.dest)) return CMD_ERROR; const Station *st = GetStation(new_order.GetDestination());
st = GetStation(new_order.dest);
if (st->owner != OWNER_NONE && !CheckOwnership(st->owner)) { if (st->owner != OWNER_NONE && !CheckOwnership(st->owner)) {
return CMD_ERROR; return CMD_ERROR;
@ -330,10 +329,9 @@ CommandCost CmdInsertOrder(TileIndex tile, uint32 flags, uint32 p1, uint32 p2)
case OT_GOTO_DEPOT: { case OT_GOTO_DEPOT: {
if (v->type == VEH_AIRCRAFT) { if (v->type == VEH_AIRCRAFT) {
const Station* st; if (!IsValidStationID(new_order.GetDestination())) return CMD_ERROR;
if (!IsValidStationID(new_order.dest)) return CMD_ERROR; const Station *st = GetStation(new_order.GetDestination());
st = GetStation(new_order.dest);
if (!CheckOwnership(st->owner) || if (!CheckOwnership(st->owner) ||
!(st->facilities & FACIL_AIRPORT) || !(st->facilities & FACIL_AIRPORT) ||
@ -342,10 +340,9 @@ CommandCost CmdInsertOrder(TileIndex tile, uint32 flags, uint32 p1, uint32 p2)
return CMD_ERROR; return CMD_ERROR;
} }
} else { } else {
const Depot* dp; if (!IsValidDepotID(new_order.GetDestination())) return CMD_ERROR;
if (!IsValidDepotID(new_order.dest)) return CMD_ERROR; const Depot *dp = GetDepot(new_order.GetDestination());
dp = GetDepot(new_order.dest);
if (!CheckOwnership(GetTileOwner(dp->xy))) return CMD_ERROR; if (!CheckOwnership(GetTileOwner(dp->xy))) return CMD_ERROR;
@ -385,12 +382,11 @@ CommandCost CmdInsertOrder(TileIndex tile, uint32 flags, uint32 p1, uint32 p2)
} }
case OT_GOTO_WAYPOINT: { case OT_GOTO_WAYPOINT: {
const Waypoint* wp;
if (v->type != VEH_TRAIN) return CMD_ERROR; if (v->type != VEH_TRAIN) return CMD_ERROR;
if (!IsValidWaypointID(new_order.dest)) return CMD_ERROR; if (!IsValidWaypointID(new_order.GetDestination())) return CMD_ERROR;
wp = GetWaypoint(new_order.dest); const Waypoint *wp = GetWaypoint(new_order.GetDestination());
if (!CheckOwnership(GetTileOwner(wp->xy))) return CMD_ERROR; if (!CheckOwnership(GetTileOwner(wp->xy))) return CMD_ERROR;
@ -769,7 +765,7 @@ CommandCost CmdModifyOrder(TileIndex tile, uint32 flags, uint32 p1, uint32 p2)
if (sel_ord >= v->num_orders) return CMD_ERROR; if (sel_ord >= v->num_orders) return CMD_ERROR;
order = GetVehicleOrder(v, sel_ord); order = GetVehicleOrder(v, sel_ord);
if ((!order->IsType(OT_GOTO_STATION) || GetStation(order->dest)->IsBuoy()) && if ((!order->IsType(OT_GOTO_STATION) || GetStation(order->GetDestination())->IsBuoy()) &&
(!order->IsType(OT_GOTO_DEPOT) || p2 == OF_UNLOAD) && (!order->IsType(OT_GOTO_DEPOT) || p2 == OF_UNLOAD) &&
(!order->IsType(OT_GOTO_WAYPOINT) || p2 != OF_NON_STOP)) { (!order->IsType(OT_GOTO_WAYPOINT) || p2 != OF_NON_STOP)) {
return CMD_ERROR; return CMD_ERROR;
@ -909,7 +905,7 @@ CommandCost CmdCloneOrder(TileIndex tile, uint32 flags, uint32 p1, uint32 p2)
FOR_VEHICLE_ORDERS(src, order) { FOR_VEHICLE_ORDERS(src, order) {
if (order->IsType(OT_GOTO_STATION)) { if (order->IsType(OT_GOTO_STATION)) {
const Station *st = GetStation(order->dest); const Station *st = GetStation(order->GetDestination());
if (IsCargoInClass(dst->cargo_type, CC_PASSENGERS)) { if (IsCargoInClass(dst->cargo_type, CC_PASSENGERS)) {
if (st->bus_stops != NULL) required_dst = st->bus_stops->xy; if (st->bus_stops != NULL) required_dst = st->bus_stops->xy;
} else { } else {
@ -1190,7 +1186,7 @@ void CheckOrders(const Vehicle* v)
} }
/* Does station have a load-bay for this vehicle? */ /* Does station have a load-bay for this vehicle? */
if (order->IsType(OT_GOTO_STATION)) { if (order->IsType(OT_GOTO_STATION)) {
const Station* st = GetStation(order->dest); const Station* st = GetStation(order->GetDestination());
TileIndex required_tile = GetStationTileForVehicle(v, st); TileIndex required_tile = GetStationTileForVehicle(v, st);
n_st++; n_st++;
@ -1251,7 +1247,7 @@ void RemoveOrderFromAllVehicles(OrderType type, DestinationID destination)
order = &v->current_order; order = &v->current_order;
if ((v->type == VEH_AIRCRAFT && order->IsType(OT_GOTO_DEPOT) ? OT_GOTO_STATION : order->GetType()) == type && if ((v->type == VEH_AIRCRAFT && order->IsType(OT_GOTO_DEPOT) ? OT_GOTO_STATION : order->GetType()) == type &&
v->current_order.dest == destination) { v->current_order.GetDestination() == destination) {
order->MakeDummy(); order->MakeDummy();
InvalidateWindow(WC_VEHICLE_VIEW, v->index); InvalidateWindow(WC_VEHICLE_VIEW, v->index);
} }
@ -1260,7 +1256,7 @@ void RemoveOrderFromAllVehicles(OrderType type, DestinationID destination)
invalidate = false; invalidate = false;
FOR_VEHICLE_ORDERS(v, order) { FOR_VEHICLE_ORDERS(v, order) {
if ((v->type == VEH_AIRCRAFT && order->IsType(OT_GOTO_DEPOT) ? OT_GOTO_STATION : order->GetType()) == type && if ((v->type == VEH_AIRCRAFT && order->IsType(OT_GOTO_DEPOT) ? OT_GOTO_STATION : order->GetType()) == type &&
order->dest == destination) { order->GetDestination() == destination) {
order->MakeDummy(); order->MakeDummy();
invalidate = true; invalidate = true;
} }
@ -1414,8 +1410,8 @@ bool ProcessOrders(Vehicle *v)
if (_patches.new_nonstop && if (_patches.new_nonstop &&
v->current_order.flags & OFB_NON_STOP && v->current_order.flags & OFB_NON_STOP &&
IsTileType(v->tile, MP_STATION) && IsTileType(v->tile, MP_STATION) &&
v->current_order.dest == GetStationIndex(v->tile)) { v->current_order.GetDestination() == GetStationIndex(v->tile)) {
v->last_station_visited = v->current_order.dest; v->last_station_visited = v->current_order.GetDestination();
UpdateVehicleTimetable(v, true); UpdateVehicleTimetable(v, true);
v->cur_order_index++; v->cur_order_index++;
} }
@ -1442,7 +1438,7 @@ bool ProcessOrders(Vehicle *v)
/* If it is unchanged, keep it. */ /* If it is unchanged, keep it. */
if (order->Equals(v->current_order) && if (order->Equals(v->current_order) &&
(v->type != VEH_SHIP || !order->IsType(OT_GOTO_STATION) || GetStation(order->dest)->dock_tile != 0)) { (v->type != VEH_SHIP || !order->IsType(OT_GOTO_STATION) || GetStation(order->GetDestination())->dock_tile != 0)) {
return false; return false;
} }
@ -1466,15 +1462,15 @@ bool ProcessOrders(Vehicle *v)
switch (order->GetType()) { switch (order->GetType()) {
case OT_GOTO_STATION: case OT_GOTO_STATION:
v->dest_tile = v->GetOrderStationLocation(order->dest); v->dest_tile = v->GetOrderStationLocation(order->GetDestination());
break; break;
case OT_GOTO_DEPOT: case OT_GOTO_DEPOT:
if (v->type != VEH_AIRCRAFT) v->dest_tile = GetDepot(order->dest)->xy; if (v->type != VEH_AIRCRAFT) v->dest_tile = GetDepot(order->GetDestination())->xy;
break; break;
case OT_GOTO_WAYPOINT: case OT_GOTO_WAYPOINT:
v->dest_tile = GetWaypoint(order->dest)->xy; v->dest_tile = GetWaypoint(order->GetDestination())->xy;
break; break;
default: default:

19
src/order_gui.cpp
View File

@ -155,7 +155,7 @@ static void DrawOrdersWindow(Window *w)
if (order != NULL) { if (order != NULL) {
switch (order->GetType()) { switch (order->GetType()) {
case OT_GOTO_STATION: case OT_GOTO_STATION:
if (!GetStation(order->dest)->IsBuoy()) break; if (!GetStation(order->GetDestination())->IsBuoy()) break;
/* Fall-through */ /* Fall-through */
case OT_GOTO_WAYPOINT: case OT_GOTO_WAYPOINT:
@ -197,12 +197,12 @@ static void DrawOrdersWindow(Window *w)
switch (order->GetType()) { switch (order->GetType()) {
case OT_DUMMY: case OT_DUMMY:
SetDParam(1, STR_INVALID_ORDER); SetDParam(1, STR_INVALID_ORDER);
SetDParam(2, order->dest); SetDParam(2, order->GetDestination());
break; break;
case OT_GOTO_STATION: case OT_GOTO_STATION:
SetDParam(1, StationOrderStrings[order->flags]); SetDParam(1, StationOrderStrings[order->flags]);
SetDParam(2, order->dest); SetDParam(2, order->GetDestination());
break; break;
case OT_GOTO_DEPOT: { case OT_GOTO_DEPOT: {
@ -210,9 +210,9 @@ static void DrawOrdersWindow(Window *w)
if (v->type == VEH_AIRCRAFT) { if (v->type == VEH_AIRCRAFT) {
s = STR_GO_TO_AIRPORT_HANGAR; s = STR_GO_TO_AIRPORT_HANGAR;
SetDParam(2, order->dest); SetDParam(2, order->GetDestination());
} else { } else {
SetDParam(2, GetDepot(order->dest)->town_index); SetDParam(2, GetDepot(order->GetDestination())->town_index);
switch (v->type) { switch (v->type) {
case VEH_TRAIN: s = (order->flags & OFB_NON_STOP) ? STR_880F_GO_NON_STOP_TO_TRAIN_DEPOT : STR_GO_TO_TRAIN_DEPOT; break; case VEH_TRAIN: s = (order->flags & OFB_NON_STOP) ? STR_880F_GO_NON_STOP_TO_TRAIN_DEPOT : STR_GO_TO_TRAIN_DEPOT; break;
@ -236,7 +236,7 @@ static void DrawOrdersWindow(Window *w)
case OT_GOTO_WAYPOINT: case OT_GOTO_WAYPOINT:
SetDParam(1, (order->flags & OFB_NON_STOP) ? STR_GO_NON_STOP_TO_WAYPOINT : STR_GO_TO_WAYPOINT); SetDParam(1, (order->flags & OFB_NON_STOP) ? STR_GO_NON_STOP_TO_WAYPOINT : STR_GO_TO_WAYPOINT);
SetDParam(2, order->dest); SetDParam(2, order->GetDestination());
break; break;
default: break; default: break;
@ -335,7 +335,6 @@ static Order GetOrderCmdFromTile(const Vehicle *v, TileIndex tile)
// not found // not found
order.Free(); order.Free();
order.dest = INVALID_STATION;
return order; return order;
} }
@ -563,9 +562,9 @@ static void OrdersWndProc(Window *w, WindowEvent *e)
TileIndex xy; TileIndex xy;
switch (ord->GetType()) { switch (ord->GetType()) {
case OT_GOTO_STATION: xy = GetStation(ord->dest)->xy ; break; case OT_GOTO_STATION: xy = GetStation(ord->GetDestination())->xy ; break;
case OT_GOTO_DEPOT: xy = (v->type == VEH_AIRCRAFT) ? GetStation(ord->dest)->xy : GetDepot(ord->dest)->xy; break; case OT_GOTO_DEPOT: xy = (v->type == VEH_AIRCRAFT) ? GetStation(ord->GetDestination())->xy : GetDepot(ord->GetDestination())->xy; break;
case OT_GOTO_WAYPOINT: xy = GetWaypoint(ord->dest)->xy; break; case OT_GOTO_WAYPOINT: xy = GetWaypoint(ord->GetDestination())->xy; break;
default: xy = 0; break; default: xy = 0; break;
} }

8
src/roadveh_cmd.cpp
View File

@ -1743,7 +1743,7 @@ again:
if (IsRoadVehFront(v) && ((IsInsideMM(v->u.road.state, RVSB_IN_ROAD_STOP, RVSB_IN_ROAD_STOP_END) && if (IsRoadVehFront(v) && ((IsInsideMM(v->u.road.state, RVSB_IN_ROAD_STOP, RVSB_IN_ROAD_STOP_END) &&
_road_veh_data_1[v->u.road.state - RVSB_IN_ROAD_STOP + (_opt.road_side << RVS_DRIVE_SIDE)] == v->u.road.frame) || _road_veh_data_1[v->u.road.state - RVSB_IN_ROAD_STOP + (_opt.road_side << RVS_DRIVE_SIDE)] == v->u.road.frame) ||
(IsInsideMM(v->u.road.state, RVSB_IN_DT_ROAD_STOP, RVSB_IN_DT_ROAD_STOP_END) && (IsInsideMM(v->u.road.state, RVSB_IN_DT_ROAD_STOP, RVSB_IN_DT_ROAD_STOP_END) &&
v->current_order.dest == GetStationIndex(v->tile) && v->current_order.GetDestination() == GetStationIndex(v->tile) &&
GetRoadStopType(v->tile) == (IsCargoInClass(v->cargo_type, CC_PASSENGERS) ? ROADSTOP_BUS : ROADSTOP_TRUCK) && GetRoadStopType(v->tile) == (IsCargoInClass(v->cargo_type, CC_PASSENGERS) ? ROADSTOP_BUS : ROADSTOP_TRUCK) &&
v->u.road.frame == RVC_DRIVE_THROUGH_STOP_FRAME))) { v->u.road.frame == RVC_DRIVE_THROUGH_STOP_FRAME))) {
@ -1820,9 +1820,9 @@ again:
if (!v->current_order.IsType(OT_GOTO_STATION)) { if (!v->current_order.IsType(OT_GOTO_STATION)) {
DEBUG(ms, 2, " current order type (%d) is not OT_GOTO_STATION", v->current_order.GetType()); DEBUG(ms, 2, " current order type (%d) is not OT_GOTO_STATION", v->current_order.GetType());
} else { } else {
if (v->current_order.dest != st->index) if (v->current_order.GetDestination() != st->index)
DEBUG(ms, 2, " current station %d is not target station in current_order.station (%d)", DEBUG(ms, 2, " current station %d is not target station in current_order.station (%d)",
st->index, v->current_order.dest); st->index, v->current_order.GetDestination());
} }
DEBUG(ms, 2, " force a slot clearing"); DEBUG(ms, 2, " force a slot clearing");
@ -1963,7 +1963,7 @@ void RoadVehicle::OnNewDay()
/* update destination */ /* update destination */
if (!(this->vehstatus & VS_STOPPED) && this->current_order.IsType(OT_GOTO_STATION) && this->u.road.slot == NULL && !(this->vehstatus & VS_CRASHED)) { if (!(this->vehstatus & VS_STOPPED) && this->current_order.IsType(OT_GOTO_STATION) && this->u.road.slot == NULL && !(this->vehstatus & VS_CRASHED)) {
Station *st = GetStation(this->current_order.dest); Station *st = GetStation(this->current_order.GetDestination());
RoadStop *rs = st->GetPrimaryRoadStop(this); RoadStop *rs = st->GetPrimaryRoadStop(this);
RoadStop *best = NULL; RoadStop *best = NULL;

4
src/ship_cmd.cpp
View File

@ -645,10 +645,10 @@ static void ShipController(Vehicle *v)
return; return;
} }
} else if (v->current_order.IsType(OT_GOTO_STATION)) { } else if (v->current_order.IsType(OT_GOTO_STATION)) {
v->last_station_visited = v->current_order.dest; v->last_station_visited = v->current_order.GetDestination();
/* Process station in the orderlist. */ /* Process station in the orderlist. */
Station *st = GetStation(v->current_order.dest); Station *st = GetStation(v->current_order.GetDestination());
if (st->facilities & FACIL_DOCK) { // ugly, ugly workaround for problem with ships able to drop off cargo at wrong stations if (st->facilities & FACIL_DOCK) { // ugly, ugly workaround for problem with ships able to drop off cargo at wrong stations
ShipArrivesAt(v, st); ShipArrivesAt(v, st);
v->BeginLoading(); v->BeginLoading();

4
src/station_cmd.cpp
View File

@ -1898,7 +1898,7 @@ bool HasStationInUse(StationID station, PlayerID player)
if (player == INVALID_PLAYER || v->owner == player) { if (player == INVALID_PLAYER || v->owner == player) {
const Order *order; const Order *order;
FOR_VEHICLE_ORDERS(v, order) { FOR_VEHICLE_ORDERS(v, order) {
if (order->IsType(OT_GOTO_STATION) && order->dest == station) { if (order->IsType(OT_GOTO_STATION) && order->GetDestination() == station) {
return true; return true;
} }
} }
@ -2437,7 +2437,7 @@ static VehicleEnterTileStatus VehicleEnter_Station(Vehicle *v, TileIndex tile, i
/* Check if the vehicle is stopping at this road stop */ /* Check if the vehicle is stopping at this road stop */
if (GetRoadStopType(tile) == (IsCargoInClass(v->cargo_type, CC_PASSENGERS) ? ROADSTOP_BUS : ROADSTOP_TRUCK) && if (GetRoadStopType(tile) == (IsCargoInClass(v->cargo_type, CC_PASSENGERS) ? ROADSTOP_BUS : ROADSTOP_TRUCK) &&
v->current_order.dest == GetStationIndex(tile)) { v->current_order.GetDestination() == GetStationIndex(tile)) {
SetBit(v->u.road.state, RVS_IS_STOPPING); SetBit(v->u.road.state, RVS_IS_STOPPING);
rs->AllocateDriveThroughBay(side); rs->AllocateDriveThroughBay(side);
} }

8
src/timetable_gui.cpp
View File

@ -120,7 +120,7 @@ static void DrawTimetableWindow(Window *w)
case OT_GOTO_STATION: case OT_GOTO_STATION:
SetDParam(0, (order->flags & OFB_NON_STOP) ? STR_880A_GO_NON_STOP_TO : STR_8806_GO_TO); SetDParam(0, (order->flags & OFB_NON_STOP) ? STR_880A_GO_NON_STOP_TO : STR_8806_GO_TO);
SetDParam(1, order->dest); SetDParam(1, order->GetDestination());
if (order->wait_time > 0) { if (order->wait_time > 0) {
SetDParam(2, STR_TIMETABLE_STAY_FOR); SetDParam(2, STR_TIMETABLE_STAY_FOR);
@ -134,9 +134,9 @@ static void DrawTimetableWindow(Window *w)
if (v->type == VEH_AIRCRAFT) { if (v->type == VEH_AIRCRAFT) {
string = STR_GO_TO_AIRPORT_HANGAR; string = STR_GO_TO_AIRPORT_HANGAR;
SetDParam(1, order->dest); SetDParam(1, order->GetDestination());
} else { } else {
SetDParam(1, GetDepot(order->dest)->town_index); SetDParam(1, GetDepot(order->GetDestination())->town_index);
switch (v->type) { switch (v->type) {
case VEH_TRAIN: string = (order->flags & OFB_NON_STOP) ? STR_880F_GO_NON_STOP_TO_TRAIN_DEPOT : STR_GO_TO_TRAIN_DEPOT; break; case VEH_TRAIN: string = (order->flags & OFB_NON_STOP) ? STR_880F_GO_NON_STOP_TO_TRAIN_DEPOT : STR_GO_TO_TRAIN_DEPOT; break;
@ -153,7 +153,7 @@ static void DrawTimetableWindow(Window *w)
case OT_GOTO_WAYPOINT: case OT_GOTO_WAYPOINT:
SetDParam(0, (order->flags & OFB_NON_STOP) ? STR_GO_NON_STOP_TO_WAYPOINT : STR_GO_TO_WAYPOINT); SetDParam(0, (order->flags & OFB_NON_STOP) ? STR_GO_NON_STOP_TO_WAYPOINT : STR_GO_TO_WAYPOINT);
SetDParam(1, order->dest); SetDParam(1, order->GetDestination());
break; break;
default: break; default: break;

7
src/train_cmd.cpp
View File

@ -2342,7 +2342,7 @@ static bool NtpCallbFindStation(TileIndex tile, TrainTrackFollowerData *ttfd, Tr
static void FillWithStationData(TrainTrackFollowerData* fd, const Vehicle* v) static void FillWithStationData(TrainTrackFollowerData* fd, const Vehicle* v)
{ {
fd->dest_coords = v->dest_tile; fd->dest_coords = v->dest_tile;
fd->station_index = v->current_order.IsType(OT_GOTO_STATION) ? v->current_order.dest : INVALID_STATION; fd->station_index = v->current_order.IsType(OT_GOTO_STATION) ? v->current_order.GetDestination() : INVALID_STATION;
} }
static const byte _initial_tile_subcoord[6][4][3] = { static const byte _initial_tile_subcoord[6][4][3] = {
@ -2658,7 +2658,6 @@ static void TrainEnterStation(Vehicle *v, StationID station)
} }
v->BeginLoading(); v->BeginLoading();
v->current_order.dest = 0;
} }
static byte AfterSetTrainPos(Vehicle *v, bool new_tile) static byte AfterSetTrainPos(Vehicle *v, bool new_tile)
@ -3559,7 +3558,7 @@ static void CheckIfTrainNeedsService(Vehicle *v)
const Depot* depot = GetDepotByTile(tfdd.tile); const Depot* depot = GetDepotByTile(tfdd.tile);
if (v->current_order.IsType(OT_GOTO_DEPOT) && if (v->current_order.IsType(OT_GOTO_DEPOT) &&
v->current_order.dest != depot->index && v->current_order.GetDestination() != depot->index &&
!Chance16(3, 16)) { !Chance16(3, 16)) {
return; return;
} }
@ -3583,7 +3582,7 @@ void Train::OnNewDay()
/* update destination */ /* update destination */
if (this->current_order.IsType(OT_GOTO_STATION)) { if (this->current_order.IsType(OT_GOTO_STATION)) {
TileIndex tile = GetStation(this->current_order.dest)->train_tile; TileIndex tile = GetStation(this->current_order.GetDestination())->train_tile;
if (tile != 0) this->dest_tile = tile; if (tile != 0) this->dest_tile = tile;
} }

6
src/vehicle.cpp
View File

@ -2033,7 +2033,7 @@ uint GenerateVehicleSortList(const Vehicle ***sort_list, uint16 *length_of_array
const Order *order; const Order *order;
FOR_VEHICLE_ORDERS(v, order) { FOR_VEHICLE_ORDERS(v, order) {
if (order->IsType(OT_GOTO_STATION) && order->dest == index) { if (order->IsType(OT_GOTO_STATION) && order->GetDestination() == index) {
if (n == *length_of_array) ExtendVehicleListSize(sort_list, length_of_array, 50); if (n == *length_of_array) ExtendVehicleListSize(sort_list, length_of_array, 50);
(*sort_list)[n++] = v; (*sort_list)[n++] = v;
break; break;
@ -2077,7 +2077,7 @@ uint GenerateVehicleSortList(const Vehicle ***sort_list, uint16 *length_of_array
const Order *order; const Order *order;
FOR_VEHICLE_ORDERS(v, order) { FOR_VEHICLE_ORDERS(v, order) {
if (order->IsType(OT_GOTO_DEPOT) && order->dest == index) { if (order->IsType(OT_GOTO_DEPOT) && order->GetDestination() == index) {
if (n == *length_of_array) ExtendVehicleListSize(sort_list, length_of_array, 25); if (n == *length_of_array) ExtendVehicleListSize(sort_list, length_of_array, 25);
(*sort_list)[n++] = v; (*sort_list)[n++] = v;
break; break;
@ -3131,7 +3131,7 @@ void Vehicle::BeginLoading()
assert(IsTileType(tile, MP_STATION) || type == VEH_SHIP); assert(IsTileType(tile, MP_STATION) || type == VEH_SHIP);
if (this->current_order.IsType(OT_GOTO_STATION) && if (this->current_order.IsType(OT_GOTO_STATION) &&
this->current_order.dest == this->last_station_visited) { this->current_order.GetDestination() == this->last_station_visited) {
/* Arriving at the ordered station. /* Arriving at the ordered station.
* Keep the load/unload flags, as we (obviously) still need them. */ * Keep the load/unload flags, as we (obviously) still need them. */
this->current_order.flags &= OFB_FULL_LOAD | OFB_UNLOAD | OFB_TRANSFER; this->current_order.flags &= OFB_FULL_LOAD | OFB_UNLOAD | OFB_TRANSFER;

12
src/vehicle_gui.cpp
View File

@ -918,9 +918,9 @@ void DrawSmallOrderList(const Vehicle *v, int x, int y)
sel--; sel--;
if (order->IsType(OT_GOTO_STATION)) { if (order->IsType(OT_GOTO_STATION)) {
if (v->type == VEH_SHIP && GetStation(order->dest)->IsBuoy()) continue; if (v->type == VEH_SHIP && GetStation(order->GetDestination())->IsBuoy()) continue;
SetDParam(0, order->dest); SetDParam(0, order->GetDestination());
DrawString(x, y, STR_A036, TC_FROMSTRING); DrawString(x, y, STR_A036, TC_FROMSTRING);
y += 6; y += 6;
@ -1954,7 +1954,7 @@ static void DrawVehicleViewWindow(Window *w)
} else { // vehicle is in a "normal" state, show current order } else { // vehicle is in a "normal" state, show current order
switch (v->current_order.GetType()) { switch (v->current_order.GetType()) {
case OT_GOTO_STATION: { case OT_GOTO_STATION: {
SetDParam(0, v->current_order.dest); SetDParam(0, v->current_order.GetDestination());
SetDParam(1, v->GetDisplaySpeed()); SetDParam(1, v->GetDisplaySpeed());
str = STR_HEADING_FOR_STATION + _patches.vehicle_speed; str = STR_HEADING_FOR_STATION + _patches.vehicle_speed;
} break; } break;
@ -1962,10 +1962,10 @@ static void DrawVehicleViewWindow(Window *w)
case OT_GOTO_DEPOT: { case OT_GOTO_DEPOT: {
if (v->type == VEH_AIRCRAFT) { if (v->type == VEH_AIRCRAFT) {
/* Aircrafts always go to a station, even if you say depot */ /* Aircrafts always go to a station, even if you say depot */
SetDParam(0, v->current_order.dest); SetDParam(0, v->current_order.GetDestination());
SetDParam(1, v->GetDisplaySpeed()); SetDParam(1, v->GetDisplaySpeed());
} else { } else {
Depot *depot = GetDepot(v->current_order.dest); Depot *depot = GetDepot(v->current_order.GetDestination());
SetDParam(0, depot->town_index); SetDParam(0, depot->town_index);
SetDParam(1, v->GetDisplaySpeed()); SetDParam(1, v->GetDisplaySpeed());
} }
@ -1982,7 +1982,7 @@ static void DrawVehicleViewWindow(Window *w)
case OT_GOTO_WAYPOINT: { case OT_GOTO_WAYPOINT: {
assert(v->type == VEH_TRAIN); assert(v->type == VEH_TRAIN);
SetDParam(0, v->current_order.dest); SetDParam(0, v->current_order.GetDestination());
str = STR_HEADING_FOR_WAYPOINT + _patches.vehicle_speed; str = STR_HEADING_FOR_WAYPOINT + _patches.vehicle_speed;
SetDParam(1, v->GetDisplaySpeed()); SetDParam(1, v->GetDisplaySpeed());
break; break;

8
src/yapf/yapf_destrail.hpp
View File

@ -88,16 +88,16 @@ public:
{ {
switch (v->current_order.GetType()) { switch (v->current_order.GetType()) {
case OT_GOTO_STATION: case OT_GOTO_STATION:
m_destTile = CalcStationCenterTile(v->current_order.dest); m_destTile = CalcStationCenterTile(v->current_order.GetDestination());
m_dest_station_id = v->current_order.dest; m_dest_station_id = v->current_order.GetDestination();
m_destTrackdirs = INVALID_TRACKDIR_BIT; m_destTrackdirs = INVALID_TRACKDIR_BIT;
break; break;
case OT_GOTO_WAYPOINT: { case OT_GOTO_WAYPOINT: {
Waypoint *wp = GetWaypoint(v->current_order.dest); Waypoint *wp = GetWaypoint(v->current_order.GetDestination());
if (wp == NULL) { if (wp == NULL) {
/* Invalid waypoint in orders! */ /* Invalid waypoint in orders! */
DEBUG(yapf, 0, "Invalid waypoint in orders == 0x%04X (train %d, player %d)", v->current_order.dest, v->unitnumber, (PlayerID)v->owner); DEBUG(yapf, 0, "Invalid waypoint in orders == 0x%04X (train %d, player %d)", v->current_order.GetDestination(), v->unitnumber, (PlayerID)v->owner);
break; break;
} }
m_destTile = wp->xy; m_destTile = wp->xy;