mirror of
https://github.com/OpenTTD/OpenTTD.git
synced 2025-03-09 15:41:15 +00:00
(svn r8896) -Fix
Most (i.e. 13 of 15) callers of GetNewVehiclePos() do not care for the return and the others can figure it out by inspecting the information returned in struct GetNewVehiclePosResult. Therefore remove the return value.
This commit is contained in:
parent
1864d28cb8
commit
696b895b77
@ -692,7 +692,8 @@ static void ShipController(Vehicle *v)
|
||||
|
||||
BeginVehicleMove(v);
|
||||
|
||||
if (GetNewVehiclePos(v, &gp)) {
|
||||
GetNewVehiclePos(v, &gp);
|
||||
if (gp.old_tile == gp.new_tile) {
|
||||
/* Staying in tile */
|
||||
if (IsShipInDepot(v)) {
|
||||
gp.x = v->x_pos;
|
||||
|
@ -2887,9 +2887,10 @@ static void TrainController(Vehicle *v, bool update_image)
|
||||
BeginVehicleMove(v);
|
||||
|
||||
GetNewVehiclePosResult gp;
|
||||
GetNewVehiclePos(v, &gp);
|
||||
if (v->u.rail.track != TRACK_BIT_WORMHOLE) {
|
||||
/* Not inside tunnel */
|
||||
if (GetNewVehiclePos(v, &gp)) {
|
||||
if (gp.old_tile == gp.new_tile) {
|
||||
/* Staying in the old tile */
|
||||
if (v->u.rail.track == TRACK_BIT_DEPOT) {
|
||||
/* Inside depot */
|
||||
@ -3039,8 +3040,6 @@ static void TrainController(Vehicle *v, bool update_image)
|
||||
}
|
||||
} else {
|
||||
/* In tunnel or on a bridge */
|
||||
GetNewVehiclePos(v, &gp);
|
||||
|
||||
if (!(v->vehstatus & VS_HIDDEN)) {
|
||||
v->cur_speed =
|
||||
min(v->cur_speed, GetBridge(GetBridgeType(v->tile))->speed);
|
||||
|
@ -2692,7 +2692,7 @@ void EndVehicleMove(Vehicle *v)
|
||||
}
|
||||
|
||||
/* returns true if staying in the same tile */
|
||||
bool GetNewVehiclePos(const Vehicle *v, GetNewVehiclePosResult *gp)
|
||||
void GetNewVehiclePos(const Vehicle *v, GetNewVehiclePosResult *gp)
|
||||
{
|
||||
static const int8 _delta_coord[16] = {
|
||||
-1,-1,-1, 0, 1, 1, 1, 0, /* x */
|
||||
@ -2706,7 +2706,6 @@ bool GetNewVehiclePos(const Vehicle *v, GetNewVehiclePosResult *gp)
|
||||
gp->y = y;
|
||||
gp->old_tile = v->tile;
|
||||
gp->new_tile = TileVirtXY(x, y);
|
||||
return gp->old_tile == gp->new_tile;
|
||||
}
|
||||
|
||||
static const Direction _new_direction_table[] = {
|
||||
|
@ -427,7 +427,7 @@ typedef struct GetNewVehiclePosResult {
|
||||
Trackdir GetVehicleTrackdir(const Vehicle* v);
|
||||
|
||||
/* returns true if staying in the same tile */
|
||||
bool GetNewVehiclePos(const Vehicle *v, GetNewVehiclePosResult *gp);
|
||||
void GetNewVehiclePos(const Vehicle *v, GetNewVehiclePosResult *gp);
|
||||
Direction GetDirectionTowards(const Vehicle* v, int x, int y);
|
||||
|
||||
#define BEGIN_ENUM_WAGONS(v) do {
|
||||
|
Loading…
Reference in New Issue
Block a user