Waiting on a semaphore must be outside of try

TheBestPessimist · TheBestPessimist · commit 9bc6a4e07265 · 2023-03-08T14:29:46.000+02:00
diff --git a/Plugins/Flow.Launcher.Plugin.Explorer/Search/Everything/EverythingAPI.cs b/Plugins/Flow.Launcher.Plugin.Explorer/Search/Everything/EverythingAPI.cs
@@ -184,9 +184,9 @@ private static void CheckAndThrowExceptionOnError()
 
         public static async Task IncrementRunCounterAsync(string fileOrFolder)
         {
+            await _semaphore.WaitAsync(TimeSpan.FromSeconds(1));
             try
             {
-                await _semaphore.WaitAsync(TimeSpan.FromSeconds(1));
                 _ = EverythingApiDllImport.Everything_IncRunCountFromFileName(fileOrFolder);
             }
             catch (Exception)

Original file line number	Diff line number	Diff line change
`@@ -184,9 +184,9 @@ private static void CheckAndThrowExceptionOnError()`
`184`	`184`
`185`	`185`	`public static async Task IncrementRunCounterAsync(string fileOrFolder)`
`186`	`186`	`{`
	`187`	`+ await _semaphore.WaitAsync(TimeSpan.FromSeconds(1));`
`187`	`188`	`try`
`188`	`189`	`{`
`189`		`- await _semaphore.WaitAsync(TimeSpan.FromSeconds(1));`
`190`	`190`	`_ = EverythingApiDllImport.Everything_IncRunCountFromFileName(fileOrFolder);`
`191`	`191`	`}`
`192`	`192`	`catch (Exception)`