Mercurial > hg > openttd
changeset 8469:066d7c3ca93d draft
(svn r12040) -Codechange: Change IsOrderListShared from a simple function to a class member(MagicBuzz).
author | belugas <belugas@openttd.org> |
---|---|
date | Sat, 02 Feb 2008 02:45:09 +0000 |
parents | d862bcc2b67b |
children | 4a0f0d97a7d9 |
files | src/autoreplace_cmd.cpp src/order.h src/order_cmd.cpp src/order_gui.cpp src/vehicle_base.h |
diffstat | 5 files changed, 12 insertions(+), 17 deletions(-) [+] |
line wrap: on
line diff
--- a/src/autoreplace_cmd.cpp +++ b/src/autoreplace_cmd.cpp @@ -215,7 +215,7 @@ DoCommand(0, (front->index << 16) | new_v->index, 1, DC_EXEC, CMD_MOVE_RAIL_VEHICLE); } else { // copy/clone the orders - DoCommand(0, (old_v->index << 16) | new_v->index, IsOrderListShared(old_v) ? CO_SHARE : CO_COPY, DC_EXEC, CMD_CLONE_ORDER); + DoCommand(0, (old_v->index << 16) | new_v->index, old_v->IsOrderListShared() ? CO_SHARE : CO_COPY, DC_EXEC, CMD_CLONE_ORDER); new_v->cur_order_index = old_v->cur_order_index; ChangeVehicleViewWindow(old_v, new_v); new_v->profit_this_year = old_v->profit_this_year;
--- a/src/order.h +++ b/src/order.h @@ -212,7 +212,6 @@ bool VehicleHasDepotOrders(const Vehicle *v); void CheckOrders(const Vehicle*); void DeleteVehicleOrders(Vehicle *v); -bool IsOrderListShared(const Vehicle *v); void AssignOrder(Order *order, Order data); bool CheckForValidOrders(const Vehicle* v);
--- a/src/order_cmd.cpp +++ b/src/order_cmd.cpp @@ -837,7 +837,7 @@ } /* make sure there are orders available */ - delta = IsOrderListShared(dst) ? src->num_orders + 1 : src->num_orders - dst->num_orders; + delta = dst->IsOrderListShared() ? src->num_orders + 1 : src->num_orders - dst->num_orders; if (!HasOrderPoolFree(delta)) return_cmd_error(STR_8831_NO_MORE_SPACE_FOR_ORDERS); @@ -939,7 +939,7 @@ if (v->name != NULL) bak->name = strdup(v->name); /* If we have shared orders, store it on a special way */ - if (IsOrderListShared(v)) { + if (v->IsOrderListShared()) { const Vehicle *u = (v->next_shared) ? v->next_shared : v->prev_shared; bak->clone = u->index; @@ -1209,7 +1209,7 @@ /* If we have a shared order-list, don't delete the list, but just remove our pointer */ - if (IsOrderListShared(v)) { + if (v->IsOrderListShared()) { Vehicle *u = v; v->orders = NULL; @@ -1257,17 +1257,6 @@ return (_patches.servint_ispercent) ? Clamp(index, MIN_SERVINT_PERCENT, MAX_SERVINT_PERCENT) : Clamp(index, MIN_SERVINT_DAYS, MAX_SERVINT_DAYS); } -/** - * - * Check if we share our orders with an other vehicle - * - * @return Returns the vehicle who has the same order - * - */ -bool IsOrderListShared(const Vehicle *v) -{ - return v->next_shared != NULL || v->prev_shared != NULL; -} /** *
--- a/src/order_gui.cpp +++ b/src/order_gui.cpp @@ -120,7 +120,7 @@ v = GetVehicle(w->window_number); - shared_orders = IsOrderListShared(v); + shared_orders = v->IsOrderListShared(); SetVScrollCount(w, v->num_orders + 1);
--- a/src/vehicle_base.h +++ b/src/vehicle_base.h @@ -461,6 +461,13 @@ * @return the first vehicle of the chain. */ inline Vehicle *First() const { return this->first; } + + /** + * Check if we share our orders with another vehicle. + * This is done by checking the previous and next pointers in the shared chain. + * @return true if there are other vehicles sharing the same order + */ + inline bool IsOrderListShared() const { return this->next_shared != NULL || this->prev_shared != NULL; }; }; /**