Enable set of nullability checks in NativeAOT CoreLib (#64730)

kant2002 · web-flow · commit d85ec4ee22de · 2022-02-09T23:28:14.000-08:00
Contributes to dotnet/runtimelab#106
diff --git a/src/coreclr/nativeaot/Directory.Build.props b/src/coreclr/nativeaot/Directory.Build.props
@@ -31,7 +31,7 @@
     <NoWarn>$(NoWarn),0419,0649,CA2249,CA1830</NoWarn>
 
     <!-- Disable nullability-related warnings -->
-    <NoWarn>$(NoWarn);CS8600;CS8602;CS8603;CS8604;CS8610;CS8618;CS8620;CS8625;CS8632;CS8765</NoWarn>
+    <NoWarn>$(NoWarn);CS8600;CS8602;CS8603;CS8604;CS8618;CS8625;CS8632;CS8765</NoWarn>
 
     <!-- we should just fix -->
     <NoWarn>$(NoWarn);CA1810;CA1823;CA1825;CA2208;SA1129;SA1205;SA1400;SA1517</NoWarn>
diff --git a/src/coreclr/nativeaot/System.Private.CoreLib/src/Internal/Reflection/Augments/ReflectionAugments.cs b/src/coreclr/nativeaot/System.Private.CoreLib/src/Internal/Reflection/Augments/ReflectionAugments.cs
@@ -138,7 +138,7 @@ public abstract object ActivatorCreateInstance(
             Type type, bool nonPublic);
         public abstract object ActivatorCreateInstance(
             [DynamicallyAccessedMembers(DynamicallyAccessedMemberTypes.PublicConstructors | DynamicallyAccessedMemberTypes.NonPublicConstructors)]
-            Type type, BindingFlags bindingAttr, Binder binder, object[] args, CultureInfo culture, object[] activationAttributes);
+            Type type, BindingFlags bindingAttr, Binder? binder, object?[]? args, CultureInfo? culture, object?[]? activationAttributes);
 
         // V2 api: Creates open or closed delegates to static or instance methods - relaxed signature checking allowed.
         public abstract Delegate CreateDelegate(Type type, object? firstArgument, MethodInfo method, bool throwOnBindFailure);
diff --git a/src/coreclr/nativeaot/System.Private.CoreLib/src/Internal/Runtime/Augments/ReflectionExecutionDomainCallbacks.cs b/src/coreclr/nativeaot/System.Private.CoreLib/src/Internal/Runtime/Augments/ReflectionExecutionDomainCallbacks.cs
@@ -26,7 +26,7 @@ namespace Internal.Runtime.Augments
     public abstract class ReflectionExecutionDomainCallbacks
     {
         // Api's that are exposed in System.Runtime but are really reflection apis.
-        public abstract Type GetType(string typeName, Func<AssemblyName, Assembly>? assemblyResolver, Func<Assembly, string, bool, Type>? typeResolver, bool throwOnError, bool ignoreCase, string defaultAssembly);
+        public abstract Type GetType(string typeName, Func<AssemblyName, Assembly?>? assemblyResolver, Func<Assembly?, string, bool, Type?>? typeResolver, bool throwOnError, bool ignoreCase, string defaultAssembly);
 
         public abstract IntPtr TryGetStaticClassConstructionContext(RuntimeTypeHandle runtimeTypeHandle);
 
diff --git a/src/coreclr/nativeaot/System.Private.CoreLib/src/Internal/Runtime/CompilerHelpers/ReflectionHelpers.cs b/src/coreclr/nativeaot/System.Private.CoreLib/src/Internal/Runtime/CompilerHelpers/ReflectionHelpers.cs
@@ -22,7 +22,7 @@ public static Type GetType(string typeName, string callingAssemblyName, bool thr
 
         // This entry is used to implement Type.GetType()'s ability to detect the calling assembly and use it as
         // a default assembly name.
-        public static Type ExtensibleGetType(string typeName, string callingAssemblyName, Func<AssemblyName, Assembly> assemblyResolver, Func<Assembly, string, bool, Type> typeResolver, bool throwOnError, bool ignoreCase)
+        public static Type ExtensibleGetType(string typeName, string callingAssemblyName, Func<AssemblyName, Assembly?> assemblyResolver, Func<Assembly?, string, bool, Type?>? typeResolver, bool throwOnError, bool ignoreCase)
         {
             return RuntimeAugments.Callbacks.GetType(typeName, assemblyResolver, typeResolver, throwOnError, ignoreCase, callingAssemblyName);
         }
diff --git a/src/coreclr/nativeaot/System.Private.CoreLib/src/System/Activator.CoreRT.cs b/src/coreclr/nativeaot/System.Private.CoreLib/src/System/Activator.CoreRT.cs
@@ -127,15 +127,15 @@ public static ObjectHandle CreateInstance(string assemblyName,
                                                   bool ignoreCase,
                                                   BindingFlags bindingAttr,
                                                   Binder? binder,
-                                                  object[]? args,
+                                                  object?[]? args,
                                                   CultureInfo? culture,
-                                                  object[]? activationAttributes)
+                                                  object?[]? activationAttributes)
         {
             throw new PlatformNotSupportedException(); // https://github.com/dotnet/corefx/issues/30845
         }
 
         [RequiresUnreferencedCode("Type and its constructor could be removed")]
-        public static ObjectHandle CreateInstance(string assemblyName, string typeName, object[]? activationAttributes)
+        public static ObjectHandle CreateInstance(string assemblyName, string typeName, object?[]? activationAttributes)
         {
             throw new PlatformNotSupportedException(); // https://github.com/dotnet/corefx/issues/30845
         }
diff --git a/src/coreclr/nativeaot/System.Private.CoreLib/src/System/Delegate.cs b/src/coreclr/nativeaot/System.Private.CoreLib/src/System/Delegate.cs
@@ -278,12 +278,12 @@ internal bool IsDynamicDelegate()
         }
 
         [DebuggerGuidedStepThroughAttribute]
-        protected virtual object DynamicInvokeImpl(object[]? args)
+        protected virtual object DynamicInvokeImpl(object?[]? args)
         {
             if (IsDynamicDelegate())
             {
                 // DynamicDelegate case
-                object result = ((Func<object[]?, object>)m_helperObject)(args);
+                object result = ((Func<object?[]?, object>)m_helperObject)(args);
                 DebugAnnotations.PreviousCallContainsDebuggerStepInCode();
                 return result;
             }
@@ -378,7 +378,7 @@ internal static bool InternalEqualTypes(object a, object b)
 
         // Returns a new delegate of the specified type whose implementation is provied by the
         // provided delegate.
-        internal static Delegate CreateObjectArrayDelegate(Type t, Func<object[], object> handler)
+        internal static Delegate CreateObjectArrayDelegate(Type t, Func<object?[], object?> handler)
         {
             EETypePtr delegateEEType;
             if (!t.TryGetEEType(out delegateEEType))
diff --git a/src/coreclr/nativeaot/System.Private.CoreLib/src/System/InvokeUtils.cs b/src/coreclr/nativeaot/System.Private.CoreLib/src/System/InvokeUtils.cs
@@ -352,7 +352,7 @@ internal static unsafe object CallDynamicInvokeMethod(
             IntPtr dynamicInvokeHelperMethod,
             IntPtr dynamicInvokeHelperGenericDictionary,
             object targetMethodOrDelegate,
-            object[]? parameters,
+            object?[]? parameters,
             BinderBundle? binderBundle,
             bool wrapInTargetInvocationException,
             bool methodToCallIsThisCall = true)
@@ -509,7 +509,7 @@ internal static ref IntPtr DynamicInvokeParamHelperRef(ref ArgSetupState argSetu
             }
         }
 
-        internal static object DynamicInvokeBoxedValuetypeReturn(out DynamicInvokeParamLookupType paramLookupType, object? boxedValuetype, object[]? parameters, int index, RuntimeTypeHandle type, DynamicInvokeParamType paramType, ref object[] nullableCopyBackObjects)
+        internal static object DynamicInvokeBoxedValuetypeReturn(out DynamicInvokeParamLookupType paramLookupType, object? boxedValuetype, object?[]? parameters, int index, RuntimeTypeHandle type, DynamicInvokeParamType paramType, ref object[] nullableCopyBackObjects)
         {
             object finalObjectToReturn = boxedValuetype;
             EETypePtr eeType = type.ToEETypePtr();
diff --git a/src/coreclr/nativeaot/System.Private.CoreLib/src/System/Reflection/Emit/ConstructorBuilder.cs b/src/coreclr/nativeaot/System.Private.CoreLib/src/System/Reflection/Emit/ConstructorBuilder.cs
@@ -114,12 +114,12 @@ public override ParameterInfo[] GetParameters()
             return default;
         }
 
-        public override object Invoke(object obj, BindingFlags invokeAttr, Binder binder, object[] parameters, CultureInfo culture)
+        public override object? Invoke(object? obj, BindingFlags invokeAttr, Binder? binder, object?[]? parameters, CultureInfo? culture)
         {
             return default;
         }
 
-        public override object Invoke(BindingFlags invokeAttr, Binder binder, object[] parameters, CultureInfo culture)
+        public override object Invoke(BindingFlags invokeAttr, Binder? binder, object?[]? parameters, CultureInfo? culture)
         {
             return default;
         }
diff --git a/src/coreclr/nativeaot/System.Private.CoreLib/src/System/Reflection/Emit/DynamicMethod.cs b/src/coreclr/nativeaot/System.Private.CoreLib/src/System/Reflection/Emit/DynamicMethod.cs
@@ -194,7 +194,7 @@ public override ParameterInfo[] GetParameters()
             return default;
         }
 
-        public override object Invoke(object obj, BindingFlags invokeAttr, Binder binder, object[] parameters, CultureInfo culture)
+        public override object? Invoke(object? obj, BindingFlags invokeAttr, Binder? binder, object?[]? parameters, CultureInfo? culture)
         {
             return default;
         }
diff --git a/src/coreclr/nativeaot/System.Private.CoreLib/src/System/Reflection/Emit/EnumBuilder.cs b/src/coreclr/nativeaot/System.Private.CoreLib/src/System/Reflection/Emit/EnumBuilder.cs
@@ -280,7 +280,7 @@ protected override bool HasElementTypeImpl()
         }
 
         [DynamicallyAccessedMembers(DynamicallyAccessedMemberTypes.All)]
-        public override object InvokeMember(string name, BindingFlags invokeAttr, Binder binder, object target, object[] args, ParameterModifier[] modifiers, System.Globalization.CultureInfo culture, string[] namedParameters)
+        public override object? InvokeMember(string name, BindingFlags invokeAttr, Binder? binder, object? target, object?[]? args, ParameterModifier[]? modifiers, Globalization.CultureInfo? culture, string[]? namedParameters)
         {
             return default;
         }
diff --git a/src/coreclr/nativeaot/System.Private.CoreLib/src/System/Reflection/Emit/GenericTypeParameterBuilder.cs b/src/coreclr/nativeaot/System.Private.CoreLib/src/System/Reflection/Emit/GenericTypeParameterBuilder.cs
@@ -326,7 +326,7 @@ protected override bool HasElementTypeImpl()
         }
 
         [DynamicallyAccessedMembers(DynamicallyAccessedMemberTypes.All)]
-        public override object InvokeMember(string name, BindingFlags invokeAttr, Binder binder, object target, object[] args, ParameterModifier[] modifiers, Globalization.CultureInfo culture, string[] namedParameters)
+        public override object? InvokeMember(string name, BindingFlags invokeAttr, Binder? binder, object? target, object?[]? args, ParameterModifier[]? modifiers, Globalization.CultureInfo? culture, string[]? namedParameters)
         {
             return default;
         }
diff --git a/src/coreclr/nativeaot/System.Private.CoreLib/src/System/Reflection/Emit/MethodBuilder.cs b/src/coreclr/nativeaot/System.Private.CoreLib/src/System/Reflection/Emit/MethodBuilder.cs
@@ -200,7 +200,7 @@ public override ParameterInfo[] GetParameters()
             return default;
         }
 
-        public override object Invoke(object obj, BindingFlags invokeAttr, Binder binder, object[] parameters, Globalization.CultureInfo culture)
+        public override object? Invoke(object? obj, BindingFlags invokeAttr, Binder? binder, object?[]? parameters, Globalization.CultureInfo? culture)
         {
             return default;
         }
diff --git a/src/coreclr/nativeaot/System.Private.CoreLib/src/System/Reflection/Emit/PropertyBuilder.cs b/src/coreclr/nativeaot/System.Private.CoreLib/src/System/Reflection/Emit/PropertyBuilder.cs
@@ -108,12 +108,12 @@ public override MethodInfo GetSetMethod(bool nonPublic)
             return default;
         }
 
-        public override object GetValue(object obj, object[] index)
+        public override object? GetValue(object? obj, object?[]? index)
         {
             return default;
         }
 
-        public override object GetValue(object obj, BindingFlags invokeAttr, Binder binder, object[] index, Globalization.CultureInfo culture)
+        public override object? GetValue(object? obj, BindingFlags invokeAttr, Binder? binder, object?[]? index, Globalization.CultureInfo? culture)
         {
             return default;
         }
@@ -143,11 +143,11 @@ public void SetSetMethod(MethodBuilder mdBuilder)
         {
         }
 
-        public override void SetValue(object obj, object value, object[] index)
+        public override void SetValue(object? obj, object? value, object?[]? index)
         {
         }
 
-        public override void SetValue(object obj, object value, BindingFlags invokeAttr, Binder binder, object[] index, Globalization.CultureInfo culture)
+        public override void SetValue(object? obj, object? value, BindingFlags invokeAttr, Binder? binder, object?[]? index, Globalization.CultureInfo? culture)
         {
         }
     }
diff --git a/src/coreclr/nativeaot/System.Private.CoreLib/src/System/Reflection/Emit/TypeBuilder.cs b/src/coreclr/nativeaot/System.Private.CoreLib/src/System/Reflection/Emit/TypeBuilder.cs
@@ -543,7 +543,7 @@ protected override bool HasElementTypeImpl()
         }
 
         [DynamicallyAccessedMembers(DynamicallyAccessedMemberTypes.All)]
-        public override object InvokeMember(string name, BindingFlags invokeAttr, Binder binder, object target, object[] args, ParameterModifier[] modifiers, Globalization.CultureInfo culture, string[] namedParameters)
+        public override object? InvokeMember(string name, BindingFlags invokeAttr, Binder? binder, object? target, object?[]? args, ParameterModifier[]? modifiers, Globalization.CultureInfo? culture, string[]? namedParameters)
         {
             return default;
         }
diff --git a/src/coreclr/nativeaot/System.Private.DisabledReflection/src/Internal/Reflection/ReflectionCoreCallbacksImplementation.cs b/src/coreclr/nativeaot/System.Private.DisabledReflection/src/Internal/Reflection/ReflectionCoreCallbacksImplementation.cs
@@ -26,7 +26,7 @@ public override object ActivatorCreateInstance(
             Type type, bool nonPublic) => throw new NotSupportedException(SR.Reflection_Disabled);
         public override object ActivatorCreateInstance(
             [DynamicallyAccessedMembers(DynamicallyAccessedMemberTypes.PublicConstructors | DynamicallyAccessedMemberTypes.NonPublicConstructors)]
-            Type type, BindingFlags bindingAttr, Binder binder, object[] args, CultureInfo culture, object[] activationAttributes) => throw new NotSupportedException(SR.Reflection_Disabled);
+            Type type, BindingFlags bindingAttr, Binder? binder, object?[]? args, CultureInfo? culture, object?[]? activationAttributes) => throw new NotSupportedException(SR.Reflection_Disabled);
         public override Delegate CreateDelegate(Type type, object firstArgument, MethodInfo method, bool throwOnBindFailure) => throw new NotSupportedException(SR.Reflection_Disabled);
         public override Delegate CreateDelegate(Type type, MethodInfo method, bool throwOnBindFailure) => throw new NotSupportedException(SR.Reflection_Disabled);
         [RequiresUnreferencedCode("The target method might be removed")]
diff --git a/src/coreclr/nativeaot/System.Private.DisabledReflection/src/Internal/Reflection/ReflectionExecutionDomainCallbacksImplementation.cs b/src/coreclr/nativeaot/System.Private.DisabledReflection/src/Internal/Reflection/ReflectionExecutionDomainCallbacksImplementation.cs
@@ -22,7 +22,7 @@ internal class ReflectionExecutionDomainCallbacksImplementation : ReflectionExec
         public override MethodBase GetMethodBaseFromStartAddressIfAvailable(IntPtr methodStartAddress) => null;
         public override Type GetNamedTypeForHandle(RuntimeTypeHandle typeHandle, bool isGenericTypeDefinition) => RuntimeTypeInfo.GetRuntimeTypeInfo(typeHandle);
         public override Type GetPointerTypeForHandle(RuntimeTypeHandle typeHandle) => RuntimeTypeInfo.GetRuntimeTypeInfo(typeHandle);
-        public override Type GetType(string typeName, Func<AssemblyName, Assembly> assemblyResolver, Func<Assembly, string, bool, Type> typeResolver, bool throwOnError, bool ignoreCase, string defaultAssembly) => throw new NotSupportedException(SR.Reflection_Disabled);
+        public override Type GetType(string typeName, Func<AssemblyName, Assembly?> assemblyResolver, Func<Assembly?, string, bool, Type?>? typeResolver, bool throwOnError, bool ignoreCase, string defaultAssembly) => throw new NotSupportedException(SR.Reflection_Disabled);
         public override RuntimeTypeHandle GetTypeHandleIfAvailable(Type type) => type.TypeHandle;
         public override bool IsReflectionBlocked(RuntimeTypeHandle typeHandle) => false;
         public override bool SupportsReflection(Type type) => false;
diff --git a/src/coreclr/nativeaot/System.Private.DisabledReflection/src/Internal/Reflection/RuntimeTypeInfo.cs b/src/coreclr/nativeaot/System.Private.DisabledReflection/src/Internal/Reflection/RuntimeTypeInfo.cs
@@ -153,7 +153,7 @@ public override Type[] GetInterfaces()
         public override PropertyInfo[] GetProperties(BindingFlags bindingAttr) => throw new NotSupportedException(SR.Reflection_Disabled);
 
         [DynamicallyAccessedMembers(DynamicallyAccessedMemberTypes.All)]
-        public override object InvokeMember(string name, BindingFlags invokeAttr, Binder binder, object target, object[] args, ParameterModifier[] modifiers, CultureInfo culture, string[] namedParameters)
+        public override object? InvokeMember(string name, BindingFlags invokeAttr, Binder? binder, object? target, object?[]? args, ParameterModifier[]? modifiers, CultureInfo? culture, string[]? namedParameters)
             => throw new NotSupportedException(SR.Reflection_Disabled);
 
         public override bool IsDefined(Type attributeType, bool inherit) => throw new NotSupportedException(SR.Reflection_Disabled);
diff --git a/src/coreclr/nativeaot/System.Private.TypeLoader/src/System.Private.TypeLoader.csproj b/src/coreclr/nativeaot/System.Private.TypeLoader/src/System.Private.TypeLoader.csproj
@@ -1,6 +1,7 @@
 <Project Sdk="Microsoft.NET.Sdk">
   <PropertyGroup>
     <NoWarn>$(NoWarn),CS1574</NoWarn>
+    <Nullable>disable</Nullable>
     <DefineConstants>TYPE_LOADER_IMPLEMENTATION;$(DefineConstants)</DefineConstants>
     <DefineConstants Condition="'$(TYPE_LOADER_TRACE)' != ''">TYPE_LOADER_TRACE;$(DefineConstants)</DefineConstants>
     <DefineConstants Condition="'$(GVM_RESOLUTION_TRACE)' != ''">GVM_RESOLUTION_TRACE;$(DefineConstants)</DefineConstants>

Original file line number	Diff line number	Diff line change
`@@ -26,7 +26,7 @@ namespace Internal.Runtime.Augments`
`26`	`26`	`public abstract class ReflectionExecutionDomainCallbacks`
`27`	`27`	`{`
`28`	`28`	`// Api's that are exposed in System.Runtime but are really reflection apis.`
`29`		`- public abstract Type GetType(string typeName, Func<AssemblyName, Assembly>? assemblyResolver, Func<Assembly, string, bool, Type>? typeResolver, bool throwOnError, bool ignoreCase, string defaultAssembly);`
	`29`	`+ public abstract Type GetType(string typeName, Func<AssemblyName, Assembly?>? assemblyResolver, Func<Assembly?, string, bool, Type?>? typeResolver, bool throwOnError, bool ignoreCase, string defaultAssembly);`
`30`	`30`
`31`	`31`	`public abstract IntPtr TryGetStaticClassConstructionContext(RuntimeTypeHandle runtimeTypeHandle);`
`32`	`32`
Original file line number	Diff line number	Diff line change
`@@ -22,7 +22,7 @@ public static Type GetType(string typeName, string callingAssemblyName, bool thr`
`22`	`22`
`23`	`23`	`// This entry is used to implement Type.GetType()'s ability to detect the calling assembly and use it as`
`24`	`24`	`// a default assembly name.`
`25`		`- public static Type ExtensibleGetType(string typeName, string callingAssemblyName, Func<AssemblyName, Assembly> assemblyResolver, Func<Assembly, string, bool, Type> typeResolver, bool throwOnError, bool ignoreCase)`
	`25`	`+ public static Type ExtensibleGetType(string typeName, string callingAssemblyName, Func<AssemblyName, Assembly?> assemblyResolver, Func<Assembly?, string, bool, Type?>? typeResolver, bool throwOnError, bool ignoreCase)`
`26`	`26`	`{`
`27`	`27`	`return RuntimeAugments.Callbacks.GetType(typeName, assemblyResolver, typeResolver, throwOnError, ignoreCase, callingAssemblyName);`
`28`	`28`	`}`
Original file line number	Diff line number	Diff line change
`@@ -127,15 +127,15 @@ public static ObjectHandle CreateInstance(string assemblyName,`
`127`	`127`	`bool ignoreCase,`
`128`	`128`	`BindingFlags bindingAttr,`
`129`	`129`	`Binder? binder,`
`130`		`- object[]? args,`
	`130`	`+ object?[]? args,`
`131`	`131`	`CultureInfo? culture,`
`132`		`- object[]? activationAttributes)`
	`132`	`+ object?[]? activationAttributes)`
`133`	`133`	`{`
`134`	`134`	`throw new PlatformNotSupportedException(); // https://github.com/dotnet/corefx/issues/30845`
`135`	`135`	`}`
`136`	`136`
`137`	`137`	`[RequiresUnreferencedCode("Type and its constructor could be removed")]`
`138`		`- public static ObjectHandle CreateInstance(string assemblyName, string typeName, object[]? activationAttributes)`
	`138`	`+ public static ObjectHandle CreateInstance(string assemblyName, string typeName, object?[]? activationAttributes)`
`139`	`139`	`{`
`140`	`140`	`throw new PlatformNotSupportedException(); // https://github.com/dotnet/corefx/issues/30845`
`141`	`141`	`}`
Original file line number	Diff line number	Diff line change
`@@ -278,12 +278,12 @@ internal bool IsDynamicDelegate()`
`278`	`278`	`}`
`279`	`279`
`280`	`280`	`[DebuggerGuidedStepThroughAttribute]`
`281`		`- protected virtual object DynamicInvokeImpl(object[]? args)`
	`281`	`+ protected virtual object DynamicInvokeImpl(object?[]? args)`
`282`	`282`	`{`
`283`	`283`	`if (IsDynamicDelegate())`
`284`	`284`	`{`
`285`	`285`	`// DynamicDelegate case`
`286`		`- object result = ((Func<object[]?, object>)m_helperObject)(args);`
	`286`	`+ object result = ((Func<object?[]?, object>)m_helperObject)(args);`
`287`	`287`	`DebugAnnotations.PreviousCallContainsDebuggerStepInCode();`
`288`	`288`	`return result;`
`289`	`289`	`}`
`@@ -378,7 +378,7 @@ internal static bool InternalEqualTypes(object a, object b)`
`378`	`378`
`379`	`379`	`// Returns a new delegate of the specified type whose implementation is provied by the`
`380`	`380`	`// provided delegate.`
`381`		`- internal static Delegate CreateObjectArrayDelegate(Type t, Func<object[], object> handler)`
	`381`	`+ internal static Delegate CreateObjectArrayDelegate(Type t, Func<object?[], object?> handler)`
`382`	`382`	`{`
`383`	`383`	`EETypePtr delegateEEType;`
`384`	`384`	`if (!t.TryGetEEType(out delegateEEType))`
Original file line number	Diff line number	Diff line change
`@@ -352,7 +352,7 @@ internal static unsafe object CallDynamicInvokeMethod(`
`352`	`352`	`IntPtr dynamicInvokeHelperMethod,`
`353`	`353`	`IntPtr dynamicInvokeHelperGenericDictionary,`
`354`	`354`	`object targetMethodOrDelegate,`
`355`		`- object[]? parameters,`
	`355`	`+ object?[]? parameters,`
`356`	`356`	`BinderBundle? binderBundle,`
`357`	`357`	`bool wrapInTargetInvocationException,`
`358`	`358`	`bool methodToCallIsThisCall = true)`
`@@ -509,7 +509,7 @@ internal static ref IntPtr DynamicInvokeParamHelperRef(ref ArgSetupState argSetu`
`509`	`509`	`}`
`510`	`510`	`}`
`511`	`511`
`512`		`- internal static object DynamicInvokeBoxedValuetypeReturn(out DynamicInvokeParamLookupType paramLookupType, object? boxedValuetype, object[]? parameters, int index, RuntimeTypeHandle type, DynamicInvokeParamType paramType, ref object[] nullableCopyBackObjects)`
	`512`	`+ internal static object DynamicInvokeBoxedValuetypeReturn(out DynamicInvokeParamLookupType paramLookupType, object? boxedValuetype, object?[]? parameters, int index, RuntimeTypeHandle type, DynamicInvokeParamType paramType, ref object[] nullableCopyBackObjects)`
`513`	`513`	`{`
`514`	`514`	`object finalObjectToReturn = boxedValuetype;`
`515`	`515`	`EETypePtr eeType = type.ToEETypePtr();`
Original file line number	Diff line number	Diff line change
`@@ -114,12 +114,12 @@ public override ParameterInfo[] GetParameters()`
`114`	`114`	`return default;`
`115`	`115`	`}`
`116`	`116`
`117`		`- public override object Invoke(object obj, BindingFlags invokeAttr, Binder binder, object[] parameters, CultureInfo culture)`
	`117`	`+ public override object? Invoke(object? obj, BindingFlags invokeAttr, Binder? binder, object?[]? parameters, CultureInfo? culture)`
`118`	`118`	`{`
`119`	`119`	`return default;`
`120`	`120`	`}`
`121`	`121`
`122`		`- public override object Invoke(BindingFlags invokeAttr, Binder binder, object[] parameters, CultureInfo culture)`
	`122`	`+ public override object Invoke(BindingFlags invokeAttr, Binder? binder, object?[]? parameters, CultureInfo? culture)`
`123`	`123`	`{`
`124`	`124`	`return default;`
`125`	`125`	`}`
Original file line number	Diff line number	Diff line change
`@@ -194,7 +194,7 @@ public override ParameterInfo[] GetParameters()`
`194`	`194`	`return default;`
`195`	`195`	`}`
`196`	`196`
`197`		`- public override object Invoke(object obj, BindingFlags invokeAttr, Binder binder, object[] parameters, CultureInfo culture)`
	`197`	`+ public override object? Invoke(object? obj, BindingFlags invokeAttr, Binder? binder, object?[]? parameters, CultureInfo? culture)`
`198`	`198`	`{`
`199`	`199`	`return default;`
`200`	`200`	`}`
Original file line number	Diff line number	Diff line change
`@@ -280,7 +280,7 @@ protected override bool HasElementTypeImpl()`
`280`	`280`	`}`
`281`	`281`
`282`	`282`	`[DynamicallyAccessedMembers(DynamicallyAccessedMemberTypes.All)]`
`283`		`- public override object InvokeMember(string name, BindingFlags invokeAttr, Binder binder, object target, object[] args, ParameterModifier[] modifiers, System.Globalization.CultureInfo culture, string[] namedParameters)`
	`283`	`+ public override object? InvokeMember(string name, BindingFlags invokeAttr, Binder? binder, object? target, object?[]? args, ParameterModifier[]? modifiers, Globalization.CultureInfo? culture, string[]? namedParameters)`
`284`	`284`	`{`
`285`	`285`	`return default;`
`286`	`286`	`}`