mirror
/
OpenTTD
mirror of https://github.com/OpenTTD/OpenTTD
1
0
Fork 0
Code Issues

(svn r17763) -Codechange: Rename 'wagon_btnstate' to 'replace_engines', and 'init_lists' to 'reset_sel_engine' in ReplaceVehicleWindow, add doxygen comments.

release/1.0
alberth 2009-10-11 12:51:15 +00:00
parent 8f090ec4a8
commit 54b8523936
1 changed files with 27 additions and 27 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

38
src/autoreplace_gui.cpp
View File

@ -99,14 +99,14 @@ void AddRemoveEngineFromAutoreplaceAndBuildWindows(VehicleType type)
* Window for the autoreplacing of vehicles.
*/
class ReplaceVehicleWindow : public Window {
EngineID sel_engine[2];
bool wagon_btnstate; ///< true means engine is selected
GUIEngineList list[2];
bool update_left;
bool update_right;
bool init_lists;
GroupID sel_group;
static RailType sel_railtype;
EngineID sel_engine[2]; ///< Selected engine left and right.
GUIEngineList list[2]; ///< Left and right list of engines.
bool replace_engines; ///< If \c true, engines are replaced, if \c false, wagons are replaced (only for trains).
bool update_left; ///< Rebuild left list.
bool update_right; ///< Rebuild right list.
bool reset_sel_engine; ///< Also reset #sel_engine while updating left and/or right (#update_left and/or #update_right) and no valid engine selected.
GroupID sel_group; ///< Group selected to replace.
static RailType sel_railtype; ///< Type of rail tracks selected.
/** Figure out if an engine should be added to a list.
* @param e The EngineID.
@ -144,7 +144,7 @@ class ReplaceVehicleWindow : public Window {
const Engine *e;
FOR_ALL_ENGINES_OF_TYPE(e, type) {
EngineID eid = e->index;
if (type == VEH_TRAIN && !GenerateReplaceRailList(eid, draw_left, this->wagon_btnstate)) continue; // special rules for trains
if (type == VEH_TRAIN && !GenerateReplaceRailList(eid, draw_left, this->replace_engines)) continue; // special rules for trains
if (draw_left) {
const GroupID selected_group = this->sel_group;
@ -172,7 +172,7 @@ class ReplaceVehicleWindow : public Window {
/* We need to rebuild the left list */
GenerateReplaceVehList(true);
this->vscroll.SetCount(this->list[0].Length());
if (this->init_lists && this->sel_engine[0] == INVALID_ENGINE && this->list[0].Length() != 0) {
if (this->reset_sel_engine && this->sel_engine[0] == INVALID_ENGINE && this->list[0].Length() != 0) {
this->sel_engine[0] = this->list[0][0];
}
}
@ -186,7 +186,7 @@ class ReplaceVehicleWindow : public Window {
} else {
GenerateReplaceVehList(false);
this->vscroll2.SetCount(this->list[1].Length());
if (this->init_lists && this->sel_engine[1] == INVALID_ENGINE && this->list[1].Length() != 0) {
if (this->reset_sel_engine && this->sel_engine[1] == INVALID_ENGINE && this->list[1].Length() != 0) {
this->sel_engine[1] = this->list[1][0];
}
}
@ -194,16 +194,16 @@ class ReplaceVehicleWindow : public Window {
/* Reset the flags about needed updates */
this->update_left = false;
this->update_right = false;
this->init_lists = false;
this->reset_sel_engine = false;
}
public:
ReplaceVehicleWindow(const WindowDesc *desc, VehicleType vehicletype, GroupID id_g) : Window(desc, vehicletype)
{
this->wagon_btnstate = true; // start with locomotives (all other vehicles will not read this bool)
this->replace_engines = true; // start with locomotives (all other vehicles will not read this bool)
this->update_left = true;
this->update_right = true;
this->init_lists = true;
this->reset_sel_engine = true;
this->sel_engine[0] = INVALID_ENGINE;
this->sel_engine[1] = INVALID_ENGINE;
@ -269,7 +269,7 @@ public:
SetDParam(1, c->settings.renew_keep_length ? STR_CONFIG_SETTING_ON : STR_CONFIG_SETTING_OFF);
/* set wagon/engine button */
SetDParam(2, this->wagon_btnstate ? STR_REPLACE_ENGINES : STR_REPLACE_WAGONS);
SetDParam(2, this->replace_engines ? STR_REPLACE_ENGINES : STR_REPLACE_WAGONS);
/* sets the colour of that art thing */
this->widget[RVW_WIDGET_TRAIN_FLUFF_LEFT].colour = _company_colours[_local_company];
@ -326,9 +326,9 @@ public:
{
switch (widget) {
case RVW_WIDGET_TRAIN_ENGINEWAGON_TOGGLE:
this->wagon_btnstate = !(this->wagon_btnstate);
this->replace_engines = !(this->replace_engines);
this->update_left = true;
this->init_lists = true;
this->reset_sel_engine = true;
this->SetDirty();
break;
@ -377,7 +377,7 @@ public:
this->sel_engine[click_side] = e;
if (click_side == 0) {
this->update_right = true;
this->init_lists = true;
this->reset_sel_engine = true;
}
this->SetDirty();
}
@ -397,7 +397,7 @@ public:
/* Rebuild the lists */
this->update_left = true;
this->update_right = true;
this->init_lists = true;
this->reset_sel_engine = true;
this->SetDirty();
}