Use primary constructor to address IDE0290 warnings

Signed-off-by: Kevin Locke <[email protected]>
kevinoid · Nov 30, 2023 · b178f82 · b178f82
1 parent a1e800e
commit b178f82
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 22 deletions.
diff --git a/NLCaseConvert/NameCapitalizer.cs b/NLCaseConvert/NameCapitalizer.cs
@@ -116,19 +116,13 @@ private static string GetPattern(Builder builder)
             "Microsoft.Design",
             "CA1034:NestedTypesShouldNotBeVisible",
             Justification = "Builder as nested type is conventional.")]
-        public class Builder
+        public class Builder(CultureInfo cultureInfo)
         {
             public Builder()
                 : this(CultureInfo.CurrentCulture)
             {
             }
 
-            public Builder(CultureInfo cultureInfo)
-            {
-                this.CultureInfo = cultureInfo
-                    ?? throw new ArgumentNullException(nameof(cultureInfo));
-            }
-
             /// <summary>
             /// Gets a pattern to match Arabic name prefixes which should not
             /// be capitalized.
@@ -244,7 +238,8 @@ public Builder(CultureInfo cultureInfo)
             public static string MixedCaseWordPattern { get; } =
                 @"(?!['`’-])[\p{L}\p{Mn}\p{Nd}'`’]*[\p{Lu}\p{Lt}][\p{L}\p{Mn}\p{Nd}'`’-]*";
 
-            public CultureInfo CultureInfo { get; }
+            public CultureInfo CultureInfo { get; } = cultureInfo
+                    ?? throw new ArgumentNullException(nameof(cultureInfo));
 
             /// <summary>
             /// Gets a pattern to match a word where all letters should be

diff --git a/NLCaseConvert/SentenceCapitalizer.cs b/NLCaseConvert/SentenceCapitalizer.cs
@@ -303,20 +303,15 @@ private static string GetPattern(Builder builder)
             "Microsoft.Design",
             "CA1034:NestedTypesShouldNotBeVisible",
             Justification = "Builder as nested type is conventional.")]
-        public class Builder
+        public class Builder(CultureInfo cultureInfo)
         {
             public Builder()
                 : this(CultureInfo.CurrentCulture)
             {
             }
 
-            public Builder(CultureInfo cultureInfo)
-            {
-                this.CultureInfo = cultureInfo
+            public CultureInfo CultureInfo { get; } = cultureInfo
                     ?? throw new ArgumentNullException(nameof(cultureInfo));
-            }
-
-            public CultureInfo CultureInfo { get; }
 
             public bool AfterParagraphBreak { get; set; }
 

diff --git a/NLCaseConvert/TitleCapitalizer.cs b/NLCaseConvert/TitleCapitalizer.cs
@@ -95,20 +95,15 @@ private static string GetPattern(Builder builder)
             "Microsoft.Design",
             "CA1034:NestedTypesShouldNotBeVisible",
             Justification = "Builder as nested type is conventional.")]
-        public class Builder
+        public class Builder(CultureInfo cultureInfo)
         {
             public Builder()
                 : this(CultureInfo.CurrentCulture)
             {
             }
 
-            public Builder(CultureInfo cultureInfo)
-            {
-                this.CultureInfo = cultureInfo
+            public CultureInfo CultureInfo { get; } = cultureInfo
                     ?? throw new ArgumentNullException(nameof(cultureInfo));
-            }
-
-            public CultureInfo CultureInfo { get; }
 
             /// <summary>
             /// Gets additional patterns to exclude from capitalization.