summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authoralberth <alberth@openttd.org>2009-10-11 11:47:21 +0000
committeralberth <alberth@openttd.org>2009-10-11 11:47:21 +0000
commit321139782d167f183fa12fa7cded4ee65d9ed23f (patch)
treefa824beac577888b064ecd914a1ee6929ee395c0 /src
parent9677e51261f794e88dc6b5722e48d786306ecb21 (diff)
downloadopenttd-321139782d167f183fa12fa7cded4ee65d9ed23f.tar.xz
(svn r17761) -Codechange: Removed some unused/unneeded variables from ReplaceVehicleWindow.
Diffstat (limited to 'src')
-rw-r--r--src/autoreplace_gui.cpp31
1 files changed, 12 insertions, 19 deletions
diff --git a/src/autoreplace_gui.cpp b/src/autoreplace_gui.cpp
index 22298fe69..60c10fb0d 100644
--- a/src/autoreplace_gui.cpp
+++ b/src/autoreplace_gui.cpp
@@ -99,9 +99,7 @@ void AddRemoveEngineFromAutoreplaceAndBuildWindows(VehicleType type)
* Window for the autoreplacing of vehicles.
*/
class ReplaceVehicleWindow : public Window {
- byte sel_index[2];
EngineID sel_engine[2];
- uint16 count[2];
bool wagon_btnstate; ///< true means engine is selected
GUIEngineList list[2];
bool update_left;
@@ -132,10 +130,9 @@ class ReplaceVehicleWindow : public Window {
/** Generate a list
- * @param w Window, that contains the list
* @param draw_left true if generating the left list, otherwise false
*/
- void GenerateReplaceVehList(Window *w, bool draw_left)
+ void GenerateReplaceVehList(bool draw_left)
{
EngineID selected_engine = INVALID_ENGINE;
VehicleType type = (VehicleType)this->window_number;
@@ -173,7 +170,7 @@ class ReplaceVehicleWindow : public Window {
if (this->update_left == true) {
/* We need to rebuild the left list */
- GenerateReplaceVehList(this, true);
+ GenerateReplaceVehList(true);
this->vscroll.SetCount(this->list[0].Length());
if (this->init_lists && this->sel_engine[0] == INVALID_ENGINE && this->list[0].Length() != 0) {
this->sel_engine[0] = this->list[0][0];
@@ -187,7 +184,7 @@ class ReplaceVehicleWindow : public Window {
this->list[1].Clear();
this->sel_engine[1] = INVALID_ENGINE;
} else {
- GenerateReplaceVehList(this, false);
+ GenerateReplaceVehList(false);
this->vscroll2.SetCount(this->list[1].Length());
if (this->init_lists && this->sel_engine[1] == INVALID_ENGINE && this->list[1].Length() != 0) {
this->sel_engine[1] = this->list[1][0];
@@ -245,28 +242,24 @@ public:
if (this->update_left || this->update_right) this->GenerateLists();
Company *c = Company::Get(_local_company);
- EngineID selected_id[2];
const GroupID selected_group = this->sel_group;
- selected_id[0] = this->sel_engine[0];
- selected_id[1] = this->sel_engine[1];
-
/* Disable the "Start Replacing" button if:
* Either list is empty
* or The selected replacement engine has a replacement (to prevent loops)
* or The right list (new replacement) has the existing replacement vehicle selected */
this->SetWidgetDisabledState(RVW_WIDGET_START_REPLACE,
- selected_id[0] == INVALID_ENGINE ||
- selected_id[1] == INVALID_ENGINE ||
- EngineReplacementForCompany(c, selected_id[1], selected_group) != INVALID_ENGINE ||
- EngineReplacementForCompany(c, selected_id[0], selected_group) == selected_id[1]);
+ this->sel_engine[0] == INVALID_ENGINE ||
+ this->sel_engine[1] == INVALID_ENGINE ||
+ EngineReplacementForCompany(c, this->sel_engine[1], selected_group) != INVALID_ENGINE ||
+ EngineReplacementForCompany(c, this->sel_engine[0], selected_group) == this->sel_engine[1]);
/* Disable the "Stop Replacing" button if:
* The left list (existing vehicle) is empty
* or The selected vehicle has no replacement set up */
this->SetWidgetDisabledState(RVW_WIDGET_STOP_REPLACE,
- selected_id[0] == INVALID_ENGINE ||
- !EngineHasReplacementForCompany(c, selected_id[0], selected_group));
+ this->sel_engine[0] == INVALID_ENGINE ||
+ !EngineHasReplacementForCompany(c, this->sel_engine[0], selected_group));
/* now the actual drawing of the window itself takes place */
SetDParam(0, STR_REPLACE_VEHICLE_TRAIN + this->window_number);
@@ -292,12 +285,12 @@ public:
this->DrawWidgets();
/* sets up the string for the vehicle that is being replaced to */
- if (selected_id[0] != INVALID_ENGINE) {
- if (!EngineHasReplacementForCompany(c, selected_id[0], selected_group)) {
+ if (this->sel_engine[0] != INVALID_ENGINE) {
+ if (!EngineHasReplacementForCompany(c, this->sel_engine[0], selected_group)) {
SetDParam(0, STR_REPLACE_NOT_REPLACING);
} else {
SetDParam(0, STR_ENGINE_NAME);
- SetDParam(1, EngineReplacementForCompany(c, selected_id[0], selected_group));
+ SetDParam(1, EngineReplacementForCompany(c, this->sel_engine[0], selected_group));
}
} else {
SetDParam(0, STR_REPLACE_NOT_REPLACING_VEHICLE_SELECTED);