Fix warnings

Author: sebastienros

BuildScripts/MakeInternal.ps1

@@ -13,6 +13,7 @@ ForEach ($file in $inputFiles)
 {
     $content = Get-Content -path $file
     $content = $content -creplace "public(?=\s+(((abstract|sealed|static|record)\s+)?(partial\s+)?class|delegate|enum|interface|struct|record))", "internal"
+    $content = ,"#pragma warning disable" + $content # $content is a list of lines, insert at the top
     $outputPath = Join-Path $outputFolder (Split-Path $file -Leaf)
     Out-File $outputPath UTF8 -InputObject $content
 }

src/FastExpressionCompiler.LightExpression/Expression.cs

@@ -25,7 +25,6 @@ THE SOFTWARE.
 
 // ReSharper disable CoVariantArrayConversion
 #nullable disable
-#pragma warning disable CS1591 // Missing XML comment for publicly visible type or member
 
 #define SUPPORTS_VISITOR
 

src/FastExpressionCompiler.LightExpression/ExpressionVisitor.cs

@@ -34,7 +34,6 @@ THE SOFTWARE.
 using FastExpressionCompiler.LightExpression.ImTools;
 
 #nullable disable
-#pragma warning disable CS1591 // Missing XML comment for publicly visible type or member
 
 namespace FastExpressionCompiler.LightExpression;
 
@@ -550,5 +549,3 @@ protected internal virtual Expression VisitRuntimeVariables(RuntimeVariablesExpr
 
     protected internal virtual Expression VisitDebugInfo(DebugInfoExpression node) => node;
 }
-
-#nullable restore

src/FastExpressionCompiler/FastExpressionCompiler.cs

@@ -664,8 +664,6 @@ public static void FreePooledParamTypes(Type[] paramTypes)
                 Interlocked.Exchange(ref _paramTypesPoolWithElem0OfLength1[paramCount - 1], paramTypes);
         }
 
-#pragma warning disable CS1591 // Missing XML comment for publicly visible type or member
-
         /// <summary>Collects the lambda info for the compilation</summary>
         public sealed class NestedLambdaInfo
         {
@@ -11436,4 +11434,3 @@ namespace System.Diagnostics.CodeAnalysis
     public sealed class UnscopedRefAttribute : Attribute { }
 }
 #endif
-#nullable restore

src/FastExpressionCompiler/ILReader.cs

@@ -1,3 +1,5 @@
+#nullable disable
+
 #if DEBUG
 // #define DEBUG_INTERNALS
 #endif
@@ -7,6 +9,7 @@
 using System.Collections;
 using System.Collections.Generic;
 using System.Diagnostics;
+using System.Globalization;
 using System.Reflection;
 using System.Reflection.Emit;
 using System.Text;
@@ -22,8 +25,6 @@ namespace FastExpressionCompiler.LightExpression.ILDecoder;
 namespace FastExpressionCompiler.ILDecoder;
 #endif
 
-#pragma warning disable CS1591 // Missing XML comment for publicly visible type or member
-
 [UnconditionalSuppressMessage("Trimming", "IL2026:Members annotated with 'RequiresUnreferencedCodeAttribute' require dynamic access otherwise can break functionality when trimming application code", Justification = "Uses reflection on internal types and is not trim-compatible.")]
 public static class ILReaderFactory
 {
@@ -109,7 +110,11 @@ public static StringBuilder ToILString(this IEnumerable<ILInstruction> ilInstruc
                     case OperandType.InlineMethod:
                         var m = (InlineMethodInstruction)il;
                         var sig = m.Method.ToString();
+#if NET
+                        var paramStart = sig.IndexOf('(', StringComparison.Ordinal);
+#else
                         var paramStart = sig.IndexOf('(');
+#endif
                         var paramList = paramStart == -1 ? "()" : sig.Substring(paramStart);
 
                         if (m.Method is MethodInfo met)
@@ -556,8 +561,12 @@ public class MethodBaseILProvider : IILProvider
 
     public MethodBaseILProvider(MethodBase method)
     {
+#if NET
+        ArgumentNullException.ThrowIfNull(method);
+#else
         if (method == null)
             throw new ArgumentNullException(nameof(method));
+#endif
 
         var methodType = method.GetType();
         if (methodType != _runtimeMethodInfoType & methodType != _runtimeConstructorInfoType)
@@ -613,21 +622,21 @@ public DynamicMethodILProvider(DynamicMethod method)
 
 public static class ILFormatter
 {
-    public static StringBuilder Int32ToHex(StringBuilder sb, int int32) => sb.Append(int32.ToString("X8"));
-    public static StringBuilder Int16ToHex(StringBuilder sb, int int16) => sb.Append(int16.ToString("X4"));
-    public static StringBuilder Int8ToHex(StringBuilder sb, int int8) => sb.Append(int8.ToString("X2"));
-    public static StringBuilder Argument(StringBuilder sb, int ordinal) => sb.Append($"V_{ordinal}");
-    public static StringBuilder Label(StringBuilder sb, int offset) => sb.Append($"IL_{offset:D4}");
+    public static StringBuilder Int32ToHex(StringBuilder sb, int int32) => sb.Append(int32.ToString("X8", CultureInfo.InvariantCulture));
+    public static StringBuilder Int16ToHex(StringBuilder sb, int int16) => sb.Append(int16.ToString("X4", CultureInfo.InvariantCulture));
+    public static StringBuilder Int8ToHex(StringBuilder sb, int int8) => sb.Append(int8.ToString("X2", CultureInfo.InvariantCulture));
+    public static StringBuilder Argument(StringBuilder sb, int ordinal) => sb.AppendFormat(CultureInfo.InvariantCulture, "V_{0}", ordinal);
+    public static StringBuilder Label(StringBuilder sb, int offset) => sb.AppendFormat(CultureInfo.InvariantCulture, "IL_{0:D4}", offset);
 
     public static StringBuilder MultipleLabels(StringBuilder sb, int[] offsets)
     {
         var length = offsets.Length;
         for (var i = 0; i < length; i++)
         {
-            sb.AppendFormat(i == 0 ? "(" : ", ");
+            sb.Append(i == 0 ? "(" : ", ");
             sb.Append(Label(sb, offsets[i]));
         }
-        sb.AppendFormat(")");
+        sb.Append(')');
         return sb;
     }
 
@@ -648,9 +657,9 @@ public static StringBuilder EscapedString(StringBuilder sb, string str)
             else if (ch == '\"')
                 sb.Append("\\\"");
             else if (ch == '\\')
-                sb.Append("\\");
+                sb.Append('\\');
             else if (ch < 0x20 || ch >= 0x7f)
-                sb.AppendFormat("\\u{0:x4}", (int)ch);
+                sb.AppendFormat(CultureInfo.InvariantCulture, "\\u{0:x4}", (int)ch);
             else
                 sb.Append(ch);
         }
@@ -663,10 +672,10 @@ public static StringBuilder SigByteArrayToString(StringBuilder sb, byte[] sig)
         var length = sig.Length;
         for (var i = 0; i < length; i++)
         {
-            sb.AppendFormat(i == 0 ? "SIG [" : " ");
+            sb.Append(i == 0 ? "SIG [" : " ");
             sb.Append(Int8ToHex(sb, sig[i]));
         }
-        sb.AppendFormat("]");
+        sb.Append(']');
         return sb;
     }
 }
@@ -820,5 +829,3 @@ public MemberInfo AsMember(int token)
 
     public byte[] AsSignature(int token) => this[token] as byte[];
 }
-
-#pragma warning restore CS1591

src/FastExpressionCompiler/ImTools.cs

@@ -1583,5 +1583,3 @@ public struct SmallSet16<K, TEq>() where TEq : struct, IEq<K>
     /// <summary>Set with 16 keys on stack and entries baked by the single array</summary> 
     public SmallMap<K, SmallMap.Entry<K>, TEq, Size16, Stack16<int>, Stack16<SmallMap.Entry<K>>, SmallMap.SingleArrayEntries<K, SmallMap.Entry<K>, TEq>> Set;
 }
-
-#nullable restore

src/FastExpressionCompiler/TestTools.cs

@@ -1,4 +1,6 @@
-using System;
+#nullable disable
+
+using System;
 using System.Linq;
 using System.Text;
 using System.Reflection;
@@ -19,18 +21,17 @@ namespace FastExpressionCompiler;
 
 using FastExpressionCompiler.ILDecoder;
 using FastExpressionCompiler.ImTools;
+using System.Globalization;
 using System.Linq.Expressions;
 #endif
 
-#pragma warning disable CS1591 // Missing XML comment for publicly visible type or member
-
 [UnconditionalSuppressMessage("ReflectionAnalysis", "IL2026:RequiresUnreferencedCode", Justification = "This is used for the testing purposes only.")]
 public static class TestTools
 {
-    public static bool AllowPrintIL = false;
-    public static bool AllowPrintCS = false;
-    public static bool AllowPrintExpression = false;
-    public static bool DisableAssertOpCodes = false;
+    public static bool AllowPrintIL;
+    public static bool AllowPrintCS;
+    public static bool AllowPrintExpression;
+    public static bool DisableAssertOpCodes;
 
     static TestTools()
     {
@@ -567,7 +568,7 @@ public static bool StartsWith(string expected, string actual,
         string expectedName = "expected",
         [CallerArgumentExpression(nameof(actual))]
         string actualName = "actual") =>
-        actual.StartsWith(expected) ? true : throw new AssertionException(
+        actual.StartsWith(expected, StringComparison.Ordinal) ? true : throw new AssertionException(
             $"Expected string `StartsWith({expectedName}, {actualName})`, but found expected `{expected.ToCode()}` is not at start of `{actual.ToCode()}`");
 }
 
@@ -628,9 +629,9 @@ class CallerMemberNameAttribute : Attribute { }
 class CallerLineNumberAttribute : Attribute { }
 
 [AttributeUsage(AttributeTargets.Parameter, Inherited = false)]
-class CallerArgumentExpression : Attribute
+class CallerArgumentExpressionAttribute : Attribute
 {
-    public CallerArgumentExpression(string parameterName) { }
+    public CallerArgumentExpressionAttribute(string parameterName) { }
 }
 #endif
 
@@ -993,7 +994,7 @@ public bool StartsWith(string expected, string actual,
         string expectedName = "<expected>",
         [CallerArgumentExpression(nameof(actual))]
         string actualName = "<actual>") =>
-        actual.StartsWith(expected) ? true : throw new AssertionException(
+        actual.StartsWith(expected, StringComparison.Ordinal) ? true : throw new AssertionException(
             $"Expected string `StartsWith({expectedName}, {actualName})`, but found expected `{expected.ToCode()}` is not at start of `{actual.ToCode()}`");
 }
 
@@ -1060,5 +1061,3 @@ public void Run<T>(T test, TestFlags flags = TestFlags.Default) where T : ITestX
         }
     }
 }
-
-#pragma warning restore CS1591