From dd108f2399a7beb197e3c1cc950d6728cd98c96b Mon Sep 17 00:00:00 2001 From: alberth Date: Sat, 26 Sep 2009 19:26:08 +0000 Subject: (svn r17650) -Codechange: Checking a condition once is enough. --- src/depot_gui.cpp | 33 +++++++++++++++------------------ 1 file changed, 15 insertions(+), 18 deletions(-) (limited to 'src') diff --git a/src/depot_gui.cpp b/src/depot_gui.cpp index 92d173488..afb0b912c 100644 --- a/src/depot_gui.cpp +++ b/src/depot_gui.cpp @@ -870,31 +870,28 @@ struct DepotWindow : Window { } } break; - case DEPOT_WIDGET_SELL: case DEPOT_WIDGET_SELL_CHAIN: - if (!this->IsWidgetDisabled(DEPOT_WIDGET_SELL) && - this->sel != INVALID_VEHICLE) { + case DEPOT_WIDGET_SELL: case DEPOT_WIDGET_SELL_CHAIN: { + if (this->IsWidgetDisabled(widget)) return; + if (this->sel == INVALID_VEHICLE) return; - if (this->IsWidgetDisabled(widget)) return; - if (this->sel == INVALID_VEHICLE) return; + this->HandleButtonClick(widget); - this->HandleButtonClick(widget); + const Vehicle *v = Vehicle::Get(this->sel); + this->sel = INVALID_VEHICLE; + this->SetDirty(); - const Vehicle *v = Vehicle::Get(this->sel); - this->sel = INVALID_VEHICLE; - this->SetDirty(); + int sell_cmd = (v->type == VEH_TRAIN && (widget == DEPOT_WIDGET_SELL_CHAIN || _ctrl_pressed)) ? 1 : 0; - int sell_cmd = (v->type == VEH_TRAIN && (widget == DEPOT_WIDGET_SELL_CHAIN || _ctrl_pressed)) ? 1 : 0; + bool is_engine = (v->type != VEH_TRAIN || Train::From(v)->IsFrontEngine()); - bool is_engine = (v->type != VEH_TRAIN || Train::From(v)->IsFrontEngine()); + if (is_engine) { + _backup_orders_tile = v->tile; + BackupVehicleOrders(v); + } - if (is_engine) { - _backup_orders_tile = v->tile; - BackupVehicleOrders(v); - } + if (!DoCommandP(v->tile, v->index, sell_cmd, GetCmdSellVeh(v->type)) && is_engine) _backup_orders_tile = 0; + } break; - if (!DoCommandP(v->tile, v->index, sell_cmd, GetCmdSellVeh(v->type)) && is_engine) _backup_orders_tile = 0; - } - break; default: this->sel = INVALID_VEHICLE; this->SetDirty(); -- cgit v1.2.3-70-g09d2