cpp/openttd-patchpack/source Changeset - r23443:d6dcf039fb2e

Changeset - r23443:d6dcf039fb2e

Parent rev.

Child rev.

[Not reviewed]

master

0 3 0

peter1138 - 5 years ago 2019-03-10 00:12:45
peter1138@openttd.org

Remove: Unnecessary virtual destructors on drop down list items.

3 files changed with 0 insertions and 9 deletions:

src/company_gui.cpp

src/toolbar_gui.cpp

src/widgets/dropdown_type.h

0 comments (0 inline, 0 general)

src/company_gui.cpp

➞

Show inline comments

@@ @@ -519,8 +519,6 @@ class DropDownListColourItem : public Dr @@
 public:
 	DropDownListColourItem(int result, bool masked) : DropDownListItem(result, masked) {}
 	virtual ~DropDownListColourItem() {}
 	StringID String() const
+	{
 		return this->result >= COLOUR_END ? STR_COLOUR_DEFAULT : _colour_dropdown[this->result];

src/toolbar_gui.cpp

➞

Show inline comments

@@ @@ -97,8 +97,6 @@ public: @@
 		this->checkmark_width = GetStringBoundingBox(STR_JUST_CHECKMARK).width + 3;
+	}
 	virtual ~DropDownListCheckedItem() {}
 	uint Width() const
+	{
 		return DropDownListStringItem::Width() + this->checkmark_width;
@@ @@ -129,8 +127,6 @@ public: @@
 		this->lock_size = GetSpriteSize(SPR_LOCK);
+	}
 	virtual ~DropDownListCompanyItem() {}
 	bool Selectable() const
+	{
 		return true;

src/widgets/dropdown_type.h

➞

Show inline comments

@@ @@ -43,7 +43,6 @@ public: @@
 	StringID string; ///< String ID of item
 	DropDownListStringItem(StringID string, int result, bool masked) : DropDownListItem(result, masked), string(string) {}
 	virtual ~DropDownListStringItem() {}
 	virtual bool Selectable() const { return true; }
 	virtual uint Width() const;
@@ @@ -61,7 +60,6 @@ public: @@
 	uint64 decode_params[10]; ///< Parameters of the string
 	DropDownListParamStringItem(StringID string, int result, bool masked) : DropDownListStringItem(string, result, masked) {}
 	virtual ~DropDownListParamStringItem() {}
 	virtual StringID String() const;
 	virtual void SetParam(uint index, uint64 value) { decode_params[index] = value; }
@@ @@ -75,7 +73,6 @@ public: @@
 	const char *raw_string;
 	DropDownListCharStringItem(const char *raw_string, int result, bool masked) : DropDownListStringItem(STR_JUST_RAW_STRING, result, masked), raw_string(raw_string) {}
 	virtual ~DropDownListCharStringItem() {}
 	virtual StringID String() const;
 };

0 comments (0 inline, 0 general)