Merge pull request #2149 from jasonmalinowski/fix-appdomain-hook

Only subscribe to AppDomain.AssemblyResolve once
OmniSharp · May 6, 2021 · f343373 · f343373
2 parents 2f1efb0 + e370afa
commit f343373
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/src/OmniSharp.Host/Services/AnalyzerAssemblyLoader.cs b/src/OmniSharp.Host/Services/AnalyzerAssemblyLoader.cs
@@ -231,7 +231,7 @@ private Assembly LoadFromPathImpl(string originalPath)
             string assemblyDirectory = CreateUniqueDirectoryForAssembly();
             string shadowCopyPath = CopyFileAndResources(originalPath, assemblyDirectory);
 
-            if (Interlocked.CompareExchange(ref _hookedAssemblyResolve, 0, 1) == 0)
+            if (Interlocked.CompareExchange(ref _hookedAssemblyResolve, value: 1, comparand: 0) == 0)
             {
                 AppDomain.CurrentDomain.AssemblyResolve += CurrentDomain_AssemblyResolve;
             }