View Issue Details

IDProjectCategoryLast Update
0025921AI War 2Gameplay IssueDec 16, 2021 5:04 pm
ReporterChris_McElligottPark Assigned ToChris_McElligottPark  
Status resolvedResolutionfixed 
Fixed in VersionBeta 3.767 Elderling Sanity 
Summary0025921: Strongly consider removing NewTimingsBeingBuilt.
DescriptionThis is something that was used extensively in the past, but I think it's more of a performance drain than anything else now. I also think that we're not actually showing it anywhere! The place that used to show this data no longer does for some reason, so removing this and just going to a different style of instrumentation is probably better.

This should lead to cleaner code, a bit of CPU speedup, and one less place to memory leak.
TagsNo tags attached.

Relationships

child of 0025884 resolvedChris_McElligottPark Parent: Todo prior to beta exit 

Activities

Chris_McElligottPark

Dec 16, 2021 5:04 pm

administrator   ~0063477

* NewTimingsBeingBuilt and similar have been removed from the codebase, as they have not been used in a really long time (a couple of years) and we have better timings-checks in place now. This will probably improve performance to a mild degree.

Issue History

Date Modified Username Field Change
Dec 14, 2021 5:04 pm Chris_McElligottPark New Issue
Dec 14, 2021 5:04 pm Chris_McElligottPark Relationship added child of 0025884
Dec 16, 2021 5:04 pm Chris_McElligottPark Assigned To => Chris_McElligottPark
Dec 16, 2021 5:04 pm Chris_McElligottPark Status new => resolved
Dec 16, 2021 5:04 pm Chris_McElligottPark Resolution open => fixed
Dec 16, 2021 5:04 pm Chris_McElligottPark Fixed in Version => Beta 3.767 Elderling Sanity
Dec 16, 2021 5:04 pm Chris_McElligottPark Note Added: 0063477