cpp/openttd-patchpack/source Changeset - r24053:47b8c8b643e3

Changeset - r24053:47b8c8b643e3

Parent rev.

Child rev.

[Not reviewed]

master

0 3 0

Jonathan G Rennison - 5 years ago 2020-01-06 20:31:57
j.g.rennison@gmail.com

Change: Only resort industry directory window on production change if necessary

3 files changed with 25 insertions and 9 deletions:

src/industry.h

src/industry_cmd.cpp

src/industry_gui.cpp

0 comments (0 inline, 0 general)

src/industry.h

➞

Show inline comments

@@ @@ -197,4 +197,12 @@ struct IndustryBuildData { @@
 extern IndustryBuildData _industry_builder;
 /** Special values for the industry list window for the data parameter of #InvalidateWindowData. */
 enum IndustryDirectoryInvalidateWindowData {
 	IDIWD_FORCE_REBUILD,
 	IDIWD_PRODUCTION_CHANGE,
 	IDIWD_FORCE_RESORT,
 };
 #endif /* INDUSTRY_H */

src/industry_cmd.cpp

➞

Show inline comments

@@ @@ -204,7 +204,7 @@ Industry::~Industry() @@
  */
 void Industry::PostDestructor(size_t index)
+{
-	InvalidateWindowData(WC_INDUSTRY_DIRECTORY, 0, 0);
+	InvalidateWindowData(WC_INDUSTRY_DIRECTORY, 0, IDIWD_FORCE_REBUILD);
+}
@@ @@ -1900,7 +1900,7 @@ static void DoCreateNewIndustry(Industry @@
 	if (GetIndustrySpec(i->type)->behaviour & INDUSTRYBEH_PLANT_ON_BUILT) {
 		for (uint j = 0; j != 50; j++) PlantRandomFarmField(i);
+	}
-	InvalidateWindowData(WC_INDUSTRY_DIRECTORY, 0, 0);
+	InvalidateWindowData(WC_INDUSTRY_DIRECTORY, 0, IDIWD_FORCE_REBUILD);
 	if (!_generating_world) PopulateStationsNearby(i);
+}
@@ @@ -2829,7 +2829,7 @@ void IndustryDailyLoop() @@
 	cur_company.Restore();
 	/* production-change */
-	InvalidateWindowData(WC_INDUSTRY_DIRECTORY, 0, 1);
+	InvalidateWindowData(WC_INDUSTRY_DIRECTORY, 0, IDIWD_PRODUCTION_CHANGE);
+}
 void IndustryMonthlyLoop()
@@ @@ -2851,7 +2851,7 @@ void IndustryMonthlyLoop() @@
 	cur_company.Restore();
 	/* production-change */
-	InvalidateWindowData(WC_INDUSTRY_DIRECTORY, 0, 1);
+	InvalidateWindowData(WC_INDUSTRY_DIRECTORY, 0, IDIWD_PRODUCTION_CHANGE);
+}

src/industry_gui.cpp

➞

Show inline comments

@@ @@ -1708,11 +1708,19 @@ public: @@
 	 */
 	void OnInvalidateData(int data = 0, bool gui_scope = true) override
+	{
 		if (data == 0) {
 			/* This needs to be done in command-scope to enforce rebuilding before resorting invalid data */
 			this->industries.ForceRebuild();
 		} else {
 			this->industries.ForceResort();
 		switch (data) {
 			case IDIWD_FORCE_REBUILD:
 				/* This needs to be done in command-scope to enforce rebuilding before resorting invalid data */
 				this->industries.ForceRebuild();
 				break;
 			case IDIWD_PRODUCTION_CHANGE:
 				if (this->industries.SortType() == 2) this->industries.ForceResort();
 				break;
 			default:
 				this->industries.ForceResort();
 				break;
+		}
+	}
 };

0 comments (0 inline, 0 general)