From ca3e8fba72de74d9770d1783cc02a3ab4a0ab379 Mon Sep 17 00:00:00 2001
From: Stephen Toub <stoub@microsoft.com>
Date: Mon, 29 Jul 2019 11:54:37 -0400
Subject: [PATCH] Re-enable CA2207 (value type declares a static cctor)
 (dotnet/corefx#39858)

Commit migrated from https://github.com/dotnet/corefx/commit/7e93ad2cd22832060851ba0e18fcb55129564e2c
---
 src/libraries/CodeAnalysis.ruleset                         |  1 -
 .../src/System/Configuration/OverrideModeSetting.cs        | 14 ++++----------
 2 files changed, 4 insertions(+), 11 deletions(-)

diff --git a/src/libraries/CodeAnalysis.ruleset b/src/libraries/CodeAnalysis.ruleset
index ecc03a3..c5660d2 100644
--- a/src/libraries/CodeAnalysis.ruleset
+++ b/src/libraries/CodeAnalysis.ruleset
@@ -57,7 +57,6 @@
     <Rule Id="CA2100" Action="None" /> <!-- Review SQL queries for security vulnerabilities -->
     <Rule Id="CA2101" Action="None" /> <!-- Specify marshaling for P/Invoke string arguments -->
     <Rule Id="CA2119" Action="None" /> <!-- Seal methods that satisfy private interfaces -->
-    <Rule Id="CA2207" Action="None" /> <!-- Initialize value type static fields inline -->
     <Rule Id="CA2208" Action="None" /> <!-- Instantiate exception arguments correctly -->
     <Rule Id="CA2211" Action="None" /> <!-- Non-constant fields should not be visible -->
     <Rule Id="CA2213" Action="None" /> <!-- Disposable Fields should be disposed -->
diff --git a/src/libraries/System.Configuration.ConfigurationManager/src/System/Configuration/OverrideModeSetting.cs b/src/libraries/System.Configuration.ConfigurationManager/src/System/Configuration/OverrideModeSetting.cs
index 985c783..c56f261 100644
--- a/src/libraries/System.Configuration.ConfigurationManager/src/System/Configuration/OverrideModeSetting.cs
+++ b/src/libraries/System.Configuration.ConfigurationManager/src/System/Configuration/OverrideModeSetting.cs
@@ -25,17 +25,11 @@ namespace System.Configuration
 
         private byte _mode;
 
-        internal static OverrideModeSetting s_sectionDefault;
-        internal static OverrideModeSetting s_locationDefault;
+        // Default for section is ALLOW
+        internal static readonly OverrideModeSetting s_sectionDefault = new OverrideModeSetting { _mode = (byte)OverrideMode.Allow };
 
-        static OverrideModeSetting()
-        {
-            // Default for section is ALLOW
-            s_sectionDefault = new OverrideModeSetting { _mode = (byte)OverrideMode.Allow };
-
-            // Default for location tags is INHERIT. Note that we do not make the value as existent in the XML or specified by the API
-            s_locationDefault = new OverrideModeSetting { _mode = (byte)OverrideMode.Inherit };
-        }
+        // Default for location tags is INHERIT. Note that we do not make the value as existent in the XML or specified by the API
+        internal static readonly OverrideModeSetting s_locationDefault = new OverrideModeSetting { _mode = (byte)OverrideMode.Inherit };
 
         internal static OverrideModeSetting CreateFromXmlReadValue(bool allowOverride)
         {
-- 
2.7.4