Slightly reduce memory turnover by using a locally allocated list for…

… the fast group prober. Another attempt to make "Delay" priority work, but the pathing needs to actually be executed before the next chore is chosen...

FastTrack/FastTrack.csproj

@@ -2,7 +2,7 @@
 <Project Sdk="Microsoft.NET.Sdk">
   <PropertyGroup>
     <AssemblyTitle>Fast Track</AssemblyTitle>
-    <FileVersion>0.15.3.0</FileVersion>
+    <FileVersion>0.15.4.0</FileVersion>
     <RootNamespace>PeterHan.FastTrack</RootNamespace>
     <Description>Optimizes Oxygen Not Included to improve performance.</Description>
     <AssemblyVersion>0.15.0.0</AssemblyVersion>

FastTrack/PathPatches/AsyncPathPatches.cs

@@ -137,7 +137,9 @@ internal void UpdateBrainGroup(AsyncBrainGroupUpdater inst, bool asyncProbe,
 				handled.Clear();
 				switch (priorityMode) {
 				case FastTrackOptions.NextChorePriority.Delay:
-					// Add priority brains to the delay list and wait until reachability pops
+					// Add priority brains to the delay list
+					// Chicken and egg problem: Need to update the brain to get a pathing
+					// update, but not drop the chore
 					if (gp != null) {
 						reachability = gp.UpdateCount;
 						if (reachability == lastReachability)

FastTrack/SensorPatches/FastGroupProber.cs

@@ -92,6 +92,11 @@ internal static void Init() {
 		/// The cells which were marked dirty during path probes.
 		/// </summary>
 		private readonly ConcurrentQueue<int> dirtyCells;
+
+		/// <summary>
+		/// The temporary list of cells that became dirty this update.
+		/// </summary>
+		private readonly IList<int> dirtyTemp;
 
 		/// <summary>
 		/// The cells which were marked dirty during path probes.
@@ -129,6 +134,7 @@ private FastGroupProber() {
 			cells = new int[Grid.CellCount];
 			destroyed = false;
 			dirtyCells = new ConcurrentQueue<int>();
+			dirtyTemp = new List<int>(32);
 			Mask = new ThreadsafePartitionerLayer("Path Updates", Grid.WidthInCells, Grid.
 				HeightInCells);
 			probers = new ConcurrentDictionary<object, ReachableCells>(2, 64);
@@ -285,7 +291,6 @@ private void ProcessLoop() {
 				d = destroyed;
 				if (!d && hit)
 					Process();
-				Interlocked.Increment(ref updateCount);
 			} while (!d);
 			// Clean up the object for real
 			probers.Clear();
@@ -310,11 +315,13 @@ internal void Remove(object prober) {
 		/// </summary>
 		internal void Update() {
 			// Trigger partitioner updates on the foreground thread
-			var dirtyList = ListPool<int, FastGroupProber>.Allocate();
 			while (dirtyCells.TryDequeue(out int cell))
-				dirtyList.Add(cell);
-			Mask.Trigger(dirtyList, this);
-			dirtyList.Recycle();
+				dirtyTemp.Add(cell);
+			if (dirtyTemp.Count > 0) {
+				Mask.Trigger(dirtyTemp, this);
+				dirtyTemp.Clear();
+			}
+			Interlocked.Increment(ref updateCount);
 			int n = toDo.Count;
 			if (n > 0 && FastTrackMod.GameRunning) {
 				if (n > MIN_PROCESS)