no longer have to wait for previous task to finish

Author: sharpjd

GPUPrefSwitcher/FileSwapper.cs

@@ -100,7 +100,7 @@ static byte[] GetHash(string str)
     /// <summary>
     /// Represents the file swap data and behavior of an AppEntry for the detailed, mistake-prone file swap system.
     /// </summary>
-    public class FileSwapper : FileSwapperData
+    public class FileSwapper : FileSwapperData //in the future you could totally refactor to this class to compose a FileSwapperData instead of inheriting
     {
         public AppEntryLibrarian AppEntryLibrarian { get; init; }
 
@@ -163,7 +163,7 @@ public async Task InitiateFileSwaps(PowerLineStatus forPowerLineStatus, AppEntry
          * The FileSwapPath state (Offline/Online) in the XML file is changed as soon as the respective copy operation 
          * completes, and a file swap does not occur if it's already in its appropriate respective state.
          * 
-         * This means that it at least shouldn't be possible for a race condition to mess up the Online/Offline state          <---(unless in the *extremely* unlikely event that the Online/Offline tracker change doesn't go through literally a single line after the file copy)
+         * This means that it at least shouldn't be possible for a race condition to mess up the Online/Offline state          <---(unless in the unlikely event that the Online/Offline tracker change doesn't go through after the file copy)
          * and its correspondence to which files are physically in the target path, and in the SettingsBank store. 
          * 
          * However, it's still possible for other undesirable behavior to occur. 

GPUPrefSwitcher/Switcher.cs

@@ -113,7 +113,9 @@ internal static void Stop()
         #endregion Initialization/End
 
         private static bool timerRunning = false;
-        private static Task runningUpdateTask;
+        //private static Task runningUpdateTask;
+
+        private static List<Task> runningUpdateTasks = new();
         /// <summary>
         /// There should be only one of this running at any time.
         /// </summary>
@@ -124,27 +126,25 @@ private static async Task BeginTimerForever()
             while (true)
             {
 
-                /*
-                 * Only ever run one of this task at once
-                 */
-
-                if (runningUpdateTask != null && runningUpdateTask.IsFaulted)
+                foreach(Task task in runningUpdateTasks)
                 {
-                    Logger.inst.ErrorLog(runningUpdateTask.Exception.ToString());
-                    throw runningUpdateTask.Exception;
-                }
+                    //TODO run a new task even if the previous one is going
+                    if (task != null && task.IsFaulted)
+                    {
+                        Logger.inst.ErrorLog(task.Exception.ToString());
+                        runningUpdateTasks.Remove(task);
+                        throw task.Exception;
+                    }
 
-                if (runningUpdateTask == Task.CompletedTask)
-                {
-                    runningUpdateTask = null;
-                }
+                    if (task == Task.CompletedTask)
+                    {
+                        runningUpdateTasks.Remove(task);
+                    }
 
-                if (runningUpdateTask == null)
-                {
-                    Task run = RunUpdateLogic();
-                    runningUpdateTask = run;
                 }
 
+                runningUpdateTasks.Add(RunUpdateLogic());
+
                 await Task.Delay(updateInterval); //move this to the beginning to make debugging easier
             }
         }
@@ -233,7 +233,7 @@ private static async Task RunUpdateLogic()
 
             prevPowerLineStatus = currentPowerLineStatus; //THIS MUST GO AT THE END
 
-            //await swaps; //we shouldn't, in case there is one that just keeps retrying forever
+            await swaps; //we shouldn't, in case there is one that just keeps retrying forever
 
             //return Task.CompletedTask;
         }