Merge pull request #145 from divya0795/fix/trykillprocess-retry-loop

fix: correct TryKillProcess retry loop condition
2026-08-28 23:01:13 +00:00 · 2026-07-21 23:22:04 +03:00
parent 3b776c52fc 7e8cadff24
commit 710e717677
1 changed files with 6 additions and 1 deletions
@@ -11,7 +11,12 @@ namespace WandEnhancer.Utils
        public static void TryKillProcess(string processName)
        {
            Process[] processes = Process.GetProcessesByName(processName);
-            for (int i = 0; processes.Length > i || i < 5; i++)
+            // Retry while any target process is still alive, capped at 5 attempts.
+            // The previous condition (processes.Length > i || i < 5) compared the
+            // process count to the loop index and, because of the "|| i < 5", always
+            // ran at least 5 iterations — sleeping ~1.25s even when the process was
+            // never running.
+            for (int i = 0; processes.Length > 0 && i < 5; i++)
            {
                foreach (var process in processes)
                {