Re-enable analyzers in project

GrabYourPitchforks · Jul 8, 2020 · c5ae79f · c5ae79f
1 parent edf3858
commit c5ae79f
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/src/coreclr/src/System.Private.CoreLib/System.Private.CoreLib.csproj b/src/coreclr/src/System.Private.CoreLib/System.Private.CoreLib.csproj
@@ -59,8 +59,7 @@
     <_FullFrameworkReferenceAssemblyPaths>$(MSBuildThisFileDirectory)/Documentation</_FullFrameworkReferenceAssemblyPaths>
     <SkipCommonResourcesIncludes>true</SkipCommonResourcesIncludes>
     <DocumentationFile>$(OutputPath)$(MSBuildProjectName).xml</DocumentationFile>
-    <!-- Turning off analyzers for now because they fail when they see calli invocations -->
-    <EnableAnalyzers>false</EnableAnalyzers>
+    <EnableAnalyzers>true</EnableAnalyzers>
   </PropertyGroup>
 
   <!-- Platform specific properties -->

diff --git a/src/coreclr/src/System.Private.CoreLib/src/System/RuntimeType.CoreCLR.cs b/src/coreclr/src/System.Private.CoreLib/src/System/RuntimeType.CoreCLR.cs
@@ -3882,7 +3882,7 @@ internal ActivatorCache(RuntimeMethodHandleInternal rmh)
                     // is implicitly public and no-ops.
 
                     _ctorIsPublic = true;
-                    _pfnCtor = &GC.KeepAlive; // no-op fn                    
+                    _pfnCtor = &GC.KeepAlive; // a convenient no-op fn
                 }
                 else
                 {