From a4548a4b9be2ce4ece5b376374e519bfae78d403 Mon Sep 17 00:00:00 2001 From: michi_cc Date: Fri, 31 Oct 2008 16:46:34 +0000 Subject: (svn r14550) -Fix [FS#2391]: Don't assert on path look-ahead when processing a not-part-of-orders depot-order. --- src/train_cmd.cpp | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) (limited to 'src') diff --git a/src/train_cmd.cpp b/src/train_cmd.cpp index 67245fe26..96fd4dd07 100644 --- a/src/train_cmd.cpp +++ b/src/train_cmd.cpp @@ -2926,8 +2926,12 @@ static Track ChooseTrainTrack(Vehicle* v, TileIndex tile, DiagDirection enterdir VehicleOrderSaver orders(v); /* If the current tile is the destination of the current order and - * a reservation was requested, advance to the next order. */ - if (v->tile == v->dest_tile || (v->current_order.IsType(OT_GOTO_STATION) && IsRailwayStationTile(v->tile) && v->current_order.GetDestination() == GetStationIndex(v->tile))) { + * a reservation was requested, advance to the next order. + * Don't advance on a depot order as depots are always safe end points + * for a path and no look-ahead is necessary. This also avoids a + * problem with depot orders not part of the order list when the + * order list itself is empty. */ + if (!v->current_order.IsType(OT_GOTO_DEPOT) && (v->tile == v->dest_tile || (v->current_order.IsType(OT_GOTO_STATION) && IsRailwayStationTile(v->tile) && v->current_order.GetDestination() == GetStationIndex(v->tile)))) { orders.SwitchToNextOrder(); } -- cgit v1.2.3-70-g09d2