cpp/openttd-patchpack/source Changeset - r3934:9868ebc2c374

Changeset - r3934:9868ebc2c374

Parent rev.

Child rev.

[Not reviewed]

master

0 1 0

peter1138 - 18 years ago 2006-06-02 13:15:50
peter1138@openttd.org

(svn r5071) - Fix (FS#184): "Erroneous train reversal on waypoints". When processing the next train order, do not even consider reversing the train if the last order was to a waypoint.

1 file changed with 10 insertions and 10 deletions:

train_cmd.c

0 comments (0 inline, 0 general)

train_cmd.c

➞

Show inline comments

@@ @@ -2347,7 +2347,7 @@ bad:; @@
 static bool ProcessTrainOrder(Vehicle *v)
+{
 	const Order *order;
-	bool result;
+	bool at_waypoint = false;
 	switch (v->current_order.type) {
 		case OT_GOTO_DEPOT:
@@ @@ -2366,6 +2366,7 @@ static bool ProcessTrainOrder(Vehicle *v @@
 	// check if we've reached the waypoint?
 	if (v->current_order.type == OT_GOTO_WAYPOINT && v->tile == v->dest_tile) {
 		v->cur_order_index++;
 		at_waypoint = true;
+	}
 	// check if we've reached a non-stop station while TTDPatch nonstop is enabled..
@@ @@ -2400,29 +2401,28 @@ static bool ProcessTrainOrder(Vehicle *v @@
 	v->dest_tile = 0;
 	result = false;
 	InvalidateVehicleOrder(v);
 	switch (order->type) {
 		case OT_GOTO_STATION:
 			if (order->station == v->last_station_visited)
 				v->last_station_visited = INVALID_STATION;
 			v->dest_tile = GetStation(order->station)->xy;
 			result = CheckReverseTrain(v);
 			break;
 		case OT_GOTO_DEPOT:
 			v->dest_tile = GetDepot(order->station)->xy;
 			result = CheckReverseTrain(v);
 			break;
 		case OT_GOTO_WAYPOINT:
 			v->dest_tile = GetWaypoint(order->station)->xy;
 			result = CheckReverseTrain(v);
 			break;
+	}
 	InvalidateVehicleOrder(v);
 	return result;
 		default:
 			return false;
+	}
 	return !at_waypoint && CheckReverseTrain(v);
+}
 static void MarkTrainDirty(Vehicle *v)

0 comments (0 inline, 0 general)