Fixed or suppressed warnings.

Author: XHunter74

DynamicCode.Test/CompilerUtilsTests.cs

@@ -1,10 +1,14 @@
-// Ignore Spelling: Utils
+// Ignore Spelling: Utils Clr csharp
 
 using Microsoft.CodeAnalysis.CSharp;
 using DynamicCode.Compiler;
 
 namespace DynamicCode.Test;
 
+#pragma warning disable CS8600 // Converting null literal or possible null value to non-nullable type.
+#pragma warning disable CS8603 // Possible null reference return.
+#pragma warning disable CS8602 // Dereference of a possibly null reference.
+#pragma warning disable CS8625 // Cannot convert null literal to non-nullable reference type.
 public class CompilerUtilsTests
 {
     [Fact(DisplayName = "ExtractSingleClassName returns class name for single class")]
@@ -103,7 +107,12 @@ private static class CompilerUtilsTestAccessor
         public static string GetClrTypeName(string csharpType)
         {
             var method = typeof(CompilerUtils).GetMethod("GetClrTypeName", System.Reflection.BindingFlags.NonPublic | System.Reflection.BindingFlags.Static);
+
             return (string)method.Invoke(null, new object[] { csharpType });
         }
     }
+#pragma warning restore CS8600 // Converting null literal or possible null value to non-nullable type.
+#pragma warning restore CS8602 // Dereference of a possibly null reference.
+#pragma warning restore CS8603 // Possible null reference return.
+#pragma warning restore CS8625 // Cannot convert null literal to non-nullable reference type.
 }

DynamicCode/Builder/DelegateTypeBuilder.cs

@@ -5,7 +5,7 @@
 /// </summary>
 public class DelegateTypeBuilder
 {
-    private readonly List<Type> _inputs = new();
+    private readonly List<Type> _inputs = [];
     private Type? _output;
 
     private DelegateTypeBuilder() { }

DynamicCode/Compiler/CompilerUtils.cs

@@ -37,8 +37,8 @@ public static string ExtractSingleClassName(SyntaxTree syntaxTree)
     /// <exception cref="MissingMethodException">Thrown if no matching method is found.</exception>
     public static string ExtractMethodNameMatchingDelegate(Type delegateType, SyntaxTree syntaxTree)
     {
-        var delegateInvoke = delegateType.GetMethod("Invoke");
-        var delegateParams = delegateInvoke.GetParameters();
+        var delegateInvoke = delegateType.GetMethod(Constants.InvokeMethodName);
+        var delegateParams = delegateInvoke!.GetParameters();
         var delegateReturn = delegateInvoke.ReturnType;
         var root = syntaxTree.GetRoot();
         var methodDecls = root.DescendantNodes().OfType<MethodDeclarationSyntax>();
@@ -56,6 +56,11 @@ public static string ExtractMethodNameMatchingDelegate(Type delegateType, Syntax
             for (int i = 0; i < parameters.Count; i++)
             {
                 var paramType = parameters[i].Type?.ToString();
+                if (paramType == null)
+                {
+                    match = false;
+                    break;
+                }
                 var clrParamType = GetClrTypeName(paramType);
                 var delegateParamType = delegateParams[i].ParameterType;
                 if (clrParamType != delegateParamType.Name && clrParamType != delegateParamType.FullName)
@@ -101,7 +106,7 @@ private static string GetClrTypeName(string csharpType)
             return csharpType;
 
         // Handle nullable types (e.g., int?)
-        if (csharpType.EndsWith("?"))
+        if (csharpType.EndsWith('?'))
         {
             var baseType = csharpType.TrimEnd('?');
             var clrBase = GetClrTypeName(baseType);

DynamicCode/Compiler/Constants.cs

@@ -3,4 +3,5 @@
 internal class Constants
 {
     internal const string AssemblyName = "DynamicAssembly";
+    internal const string InvokeMethodName = "Invoke";
 }