diff --git a/roadveh_cmd.c b/roadveh_cmd.c --- a/roadveh_cmd.c +++ b/roadveh_cmd.c @@ -311,13 +311,7 @@ static Depot *FindClosestRoadDepot(Vehic if (_patches.new_pathfinding_all) { NPFFoundTargetData ftd; /* See where we are now */ - byte trackdir; - if (IsRoadStationTile(tile)) - /* if we are in a station, simulate leaving the station (since - * v->direction won't contain anything usefule than */ - trackdir = _dir_to_diag_trackdir[GetRoadStationDir(tile)]; - else - trackdir = GetVehicleTrackdir(v); + byte trackdir = GetVehicleTrackdir(v); ftd = NPFRouteToDepotBreadthFirst(v->tile, trackdir, TRANSPORT_ROAD, v->owner); if (ftd.best_bird_dist == 0) @@ -1665,7 +1659,7 @@ void OnNewDay_RoadVeh(Vehicle *v) } //We do not have a slot, so make one - if (v->u.road.slot == NULL && rs != NULL && IsTileType(v->tile, MP_STREET)) { + if (v->u.road.slot == NULL && rs != NULL) { //first we need to find out how far our stations are away. DEBUG(ms, 2) ("Multistop: Attempting to obtain a slot for vehicle %d at station %d (0x%x)", v->unitnumber, st->index, st->xy); diff --git a/train_cmd.c b/train_cmd.c --- a/train_cmd.c +++ b/train_cmd.c @@ -1298,14 +1298,16 @@ static bool TrainFindDepotEnumProc(uint return length >= tfdd->best_length; } -// returns the tile of a depot to goto to. The given vehicle must be on track, -// so not crashed, in a depot, etc. +// returns the tile of a depot to goto to. The given vehicle must not be +// crashed! static TrainFindDepotData FindClosestTrainDepot(Vehicle *v) { int i; TrainFindDepotData tfdd; uint tile = v->tile; + assert(!(v->vehstatus & VS_CRASHED)); + tfdd.owner = v->owner; tfdd.best_length = (uint)-1; diff --git a/vehicle.c b/vehicle.c --- a/vehicle.c +++ b/vehicle.c @@ -17,6 +17,8 @@ #include "debug.h" #include "npf.h" #include "vehicle_gui.h" +#include "depot.h" +#include "station.h" #define INVALID_COORD (-0x8000) #define GEN_HASH(x,y) (((x & 0x1F80)>>7) + ((y & 0xFC0))) @@ -1714,11 +1716,15 @@ byte GetDirectionTowards(Vehicle *v, int byte GetVehicleTrackdir(const Vehicle* v) { + if (v->vehstatus & VS_CRASHED) + return 0xff; + switch(v->type) { case VEH_Train: if (v->u.rail.track == 0x80) - return 0xFF; /* Train in depot */ + /* We'll assume the train is facing outwards */ + return _dir_to_diag_trackdir[GetDepotDirection(v->tile, TRANSPORT_RAIL)]; /* Train in depot */ else if (v->u.rail.track == 0x40) /* train in tunnel, so just use his direction and assume a diagonal track */ return _dir_to_diag_trackdir[(v->direction>>1)&3]; @@ -1726,12 +1732,17 @@ byte GetVehicleTrackdir(const Vehicle* v return _track_direction_to_trackdir[FIND_FIRST_BIT(v->u.rail.track)][v->direction]; case VEH_Ship: if (v->u.ship.state == 0x80) - return 0xFF; /* Ship in depot */ + /* We'll assume the ship is facing outwards */ + return _dir_to_diag_trackdir[GetDepotDirection(v->tile, TRANSPORT_WATER)]; /* Ship in depot */ else return _track_direction_to_trackdir[FIND_FIRST_BIT(v->u.ship.state)][v->direction]; case VEH_Road: if (v->u.road.state == 254) - return 0xFF; /* Road vehicle in depot */ + /* We'll assume the road vehicle is facing outwards */ + return _dir_to_diag_trackdir[GetDepotDirection(v->tile, TRANSPORT_ROAD)]; /* Road vehicle in depot */ + else if (IsRoadStationTile(v->tile)) + /* We'll assume the road vehicle is facing outwards */ + return _dir_to_diag_trackdir[GetRoadStationDir(v->tile)]; /* Road vehicle in a station */ else return _dir_to_diag_trackdir[(v->direction>>1)&3]; case VEH_Aircraft: