(svn r22332) -Fix: When inserting automatic orders, do not create consecutive duplicate orders.

This commit is contained in:
frosch 2011-04-16 17:18:04 +00:00
parent 8dd2f413bd
commit 74069dbda3

View File

@ -1861,11 +1861,17 @@ void Vehicle::BeginLoading()
* to show that we are stopping here, but only do that if the order * to show that we are stopping here, but only do that if the order
* list isn't empty. */ * list isn't empty. */
Order *in_list = this->GetOrder(this->cur_auto_order_index); Order *in_list = this->GetOrder(this->cur_auto_order_index);
if (in_list != NULL && this->orders.list->GetNumOrders() < MAX_VEH_ORDER_ID && if (in_list != NULL &&
(!in_list->IsType(OT_AUTOMATIC) || (!in_list->IsType(OT_AUTOMATIC) ||
in_list->GetDestination() != this->last_station_visited) && in_list->GetDestination() != this->last_station_visited)) {
!suppress_automatic_orders && /* Do not create consecutive duplicates of automatic orders */
Order::CanAllocateItem()) { Order *prev_order = this->cur_auto_order_index > 0 ? this->GetOrder(this->cur_auto_order_index - 1) : NULL;
if (prev_order == NULL ||
(!prev_order->IsType(OT_AUTOMATIC) && !prev_order->IsType(OT_GOTO_STATION)) ||
prev_order->GetDestination() != this->last_station_visited) {
if (!suppress_automatic_orders && this->orders.list->GetNumOrders() < MAX_VEH_ORDER_ID && Order::CanAllocateItem()) {
/* Insert new automatic order */
Order *auto_order = new Order(); Order *auto_order = new Order();
auto_order->MakeAutomatic(this->last_station_visited); auto_order->MakeAutomatic(this->last_station_visited);
InsertOrder(this, auto_order, this->cur_auto_order_index); InsertOrder(this, auto_order, this->cur_auto_order_index);
@ -1876,6 +1882,8 @@ void Vehicle::BeginLoading()
uint16 &gv_flags = this->GetGroundVehicleFlags(); uint16 &gv_flags = this->GetGroundVehicleFlags();
ClrBit(gv_flags, GVF_SUPPRESS_AUTOMATIC_ORDERS); ClrBit(gv_flags, GVF_SUPPRESS_AUTOMATIC_ORDERS);
} }
}
}
this->current_order.MakeLoading(false); this->current_order.MakeLoading(false);
} }