Can these Reflection / Reflection.Emit bugfixes be backported from .NET Core to .NET Framework?

[stakx]

I've recently submitted several bugfixes for Reflection and Reflection.Emit which I would love to see backported to the .NET Framework. The goal is to increase the accuracy of dynamic proxying of types (with e.g. DynamicProxy) which would directly benefit mocking libraries (e.g. Moq, NSubstitute, FakeItEasy) and DI containers (e.g. Castle Windsor, Autofac).

I suspect that there is significant code share between the CLR and the CoreCLR in the Reflection facilities, since both runtimes had the same bugs, so backporting would seem generally feasible.

Could the following changes from CoreCLR please be backported to the .NET Framework?:

Reflection: Fix FormatException when querying ParameterInfo.DefaultValue of optional DateTime parameter

• bug report: https://github.com/dotnet/corefx/issues/26164
• implementation: Reflection: Fix FormatException when querying ParameterInfo.DefaultValue of optional DateTime parameter dotnet/coreclr#17877
• tests: Reflection: Add test verifying ParameterInfo.DefaultValue doesn't throw for optional DateTime parameter dotnet/corefx#29533

Reflection.Emit: Allow ParameterBuilder.SetConstant(null) for value-typed parameters

• bug report: https://github.com/dotnet/corefx/issues/26184
• implementation: Reflection.Emit: Allow ParameterBuilder.SetConstant(null) for value-typed parameters dotnet/coreclr#17887 + [WIP] Fix failing System.Reflection.Emit tests dotnet/coreclr#17915 (amendment)
• tests: Reflection.Emit: Add test for ParameterBuilder.SetConstant(null) dotnet/corefx#29532

Reflection: Fix FormatException when querying ParameterInfo.DefaultValue of optional (closed generic) enum parameter

• bug report: https://github.com/dotnet/corefx/issues/29570
• implementation: Reflection: Fix FormatException when querying ParameterInfo.DefaultValue of optional (closed generic) enum parameter dotnet/coreclr#17917

Reflection: Allow ParameterBuilder.SetConstant(nonNullValue) for nullable enum parameters

• bug report: https://github.com/dotnet/coreclr/issues/17893
• implementation: Reflection: Allow ParameterBuilder.SetConstant(nonNullValue) for nullable enum parameters dotnet/coreclr#17977
• tests: Reflection & Reflection.Emit: Add tests for nullable enum parameters dotnet/corefx#29676

If I am asking in the wrong place, please let me know how to proceed in order to advance this request.

[AlexGhiondea]

@stakx thanks for your help fixing those issues!

I am going to mark the issues so that we consider them for the next version of .NET Framework.

[stakx]

@AlexGhiondea: Thank you, that is great to hear and much appreciated! 👍

I assume that the labels you've added will suffice for further tracking, so I am closing this issue.

Thank you!