style: lint and minor code cleanup

Author: hogimn

SQL.Formatter.Test/FormatConfigTest.cs

@@ -6,13 +6,13 @@ namespace SQL.Formatter.Test
 {
     public class FormatConfigTest
     {
-        private static readonly SqlFormatter.Formatter Formatter = SqlFormatter.Of(Dialect.StandardSql);
+        private static readonly SqlFormatter.Formatter s_formatter = SqlFormatter.Of(Dialect.StandardSql);
 
         [Fact]
         public void Upper_FormatsKeywordsInUppercase()
         {
             var cfg = FormatConfig.Builder().Case(CaseTypes.UPPER).Build();
-            var result = Formatter.Format("select id from users where id = 1", cfg);
+            var result = s_formatter.Format("select id from users where id = 1", cfg);
 
             Assert.Equal(
                 "SELECT\n"
@@ -30,7 +30,7 @@ public void Uppercase_FormatsKeywordsInUppercase()
 #pragma warning disable CS0618
             var cfg = FormatConfig.Builder().Uppercase(true).Build();
 #pragma warning restore CS0618
-            var result = Formatter.Format("select id from users where id = 1", cfg);
+            var result = s_formatter.Format("select id from users where id = 1", cfg);
 
             Assert.Equal(
                 "SELECT\n"
@@ -48,7 +48,7 @@ public void Uppercase_False_PreservesKeywordCase()
 #pragma warning disable CS0618
             var cfg = FormatConfig.Builder().Uppercase(false).Build();
 #pragma warning restore CS0618
-            var result = Formatter.Format("select id FROM users", cfg);
+            var result = s_formatter.Format("select id FROM users", cfg);
 
             Assert.Equal(
                 "select\n"
@@ -62,7 +62,7 @@ public void Uppercase_False_PreservesKeywordCase()
         public void Lower_FormatsKeywordsInLowercase()
         {
             var cfg = FormatConfig.Builder().Case(CaseTypes.LOWER).Build();
-            var result = Formatter.Format("SELECT id FROM Users WHERE id = 1", cfg);
+            var result = s_formatter.Format("SELECT id FROM Users WHERE id = 1", cfg);
 
             Assert.Equal(
                 "select\n"
@@ -104,7 +104,7 @@ public void Upper_ClearsLower_FormatsUppercase()
                 .Case(CaseTypes.UPPER)
                 .Build();
 
-            var result = Formatter.Format("select id from users", cfg);
+            var result = s_formatter.Format("select id from users", cfg);
 
             Assert.Equal(
                 "SELECT\n"
@@ -122,7 +122,7 @@ public void Lower_ClearsUpper_FormatsLowercase()
                 .Case(CaseTypes.LOWER)
                 .Build();
 
-            var result = Formatter.Format("SELECT id FROM users", cfg);
+            var result = s_formatter.Format("SELECT id FROM users", cfg);
 
             Assert.Equal(
                 "select\n"
@@ -136,7 +136,7 @@ public void Lower_ClearsUpper_FormatsLowercase()
         public void NeitherUppercaseNorLowercase_PreservesKeywordCase()
         {
             var cfg = FormatConfig.Builder().Build();
-            var result = Formatter.Format("SeLeCt id FrOm users", cfg);
+            var result = s_formatter.Format("SeLeCt id FrOm users", cfg);
 
             Assert.Equal(
                 "SeLeCt\n"
@@ -150,7 +150,7 @@ public void NeitherUppercaseNorLowercase_PreservesKeywordCase()
         public void None_PreservesKeywordCase()
         {
             var cfg = FormatConfig.Builder().Case(CaseTypes.NONE).Build();
-            var result = Formatter.Format("SeLeCt id FrOm users", cfg);
+            var result = s_formatter.Format("SeLeCt id FrOm users", cfg);
 
             Assert.Equal(
                 "SeLeCt\n"

SQL.Formatter/Core/CaseTypes.cs

@@ -1,8 +1,4 @@
-using System;
-using System.Collections.Generic;
-using System.Text;
-
-namespace SQL.Formatter.Core
+namespace SQL.Formatter.Core
 {
     public enum CaseTypes
     {

SQL.Formatter/Core/FormatConfig.cs

@@ -24,16 +24,16 @@ public FormatConfig(
             Params parameters,
             bool uppercase,
             int linesBetweenQueries,
-            bool skipWhitespaceNearBlockParentheses) 
-            
+            bool skipWhitespaceNearBlockParentheses)
+
             : this(
-                  indent, 
-                  maxColumnLength, 
-                  parameters, 
-                  uppercase ? CaseTypes.UPPER : CaseTypes.NONE, 
-                  linesBetweenQueries, 
+                  indent,
+                  maxColumnLength,
+                  parameters,
+                  uppercase ? CaseTypes.UPPER : CaseTypes.NONE,
+                  linesBetweenQueries,
                   skipWhitespaceNearBlockParentheses)
-        {            
+        {
         }
 
         public FormatConfig(
@@ -55,7 +55,6 @@ public FormatConfig(
             SkipWhitespaceNearBlockParentheses = skipWhitespaceNearBlockParentheses;
         }
 
-
         public static FormatConfigBuilder Builder()
         {
             return new FormatConfigBuilder();