diff options
author | peter1138 <peter1138@openttd.org> | 2008-09-14 08:47:18 +0000 |
---|---|---|
committer | peter1138 <peter1138@openttd.org> | 2008-09-14 08:47:18 +0000 |
commit | 45ce441a5503626757105701c6095b3ea08a08ca (patch) | |
tree | 0b7c150df3d0680abc1dc1d19284d26bfefde937 | |
parent | 54d1b499813d8c6f1bab48e1dd00dc624d095558 (diff) | |
download | openttd-45ce441a5503626757105701c6095b3ea08a08ca.tar.xz |
(svn r14317) -Fix (r13662) [FS#2298]: Order distance processing for conditional orders on ships treated v->orders as an array, and didn't use the correct next order.
-rw-r--r-- | src/order_cmd.cpp | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/src/order_cmd.cpp b/src/order_cmd.cpp index eefebf3ba..0c7849dc1 100644 --- a/src/order_cmd.cpp +++ b/src/order_cmd.cpp @@ -304,8 +304,10 @@ static uint GetOrderDistance(const Order *prev, const Order *cur, const Vehicle if (conditional_depth > v->num_orders) return 0; conditional_depth++; - return max(GetOrderDistance(prev, &v->orders[cur->GetConditionSkipToOrder()], v, conditional_depth), - GetOrderDistance(prev, (prev + 1 == &v->orders[v->num_orders]) ? v->orders : (prev + 1), v, conditional_depth)); + + int dist1 = GetOrderDistance(prev, GetVehicleOrder(v, cur->GetConditionSkipToOrder()), v, conditional_depth); + int dist2 = GetOrderDistance(prev, cur->next == NULL ? v->orders : cur->next, v, conditional_depth); + return max(dist1, dist2); } return DistanceManhattan(GetOrderLocation(*prev), GetOrderLocation(*cur)); |