We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
System.Dynamic.Utils.DelegateHelpers
UnsafeAccessorAttribute
As discussed in: #90519 both:
DynamicDelegateLightup
runtime/src/libraries/System.Linq.Expressions/src/System/Dynamic/Utils/DelegateHelpers.cs
Line 20 in b0d4502
ForceAllowDynamicCodeLightup
Line 31 in b0d4502
should utilize UnsafeAccessorAttribute to simplify the logic of accessing the internal static methods in Delegate and AssemblyBuilder respectfully.
Delegate
AssemblyBuilder
This would additionally remove the need for Internal.Runtime.Augments.DynamicDelegateAugments forwarder and simplify the code in general.
Internal.Runtime.Augments.DynamicDelegateAugments
For reference: similar refactoring has been done in: #89706
The text was updated successfully, but these errors were encountered:
Tagging subscribers to this area: @cston See info in area-owners.md if you want to be subscribed.
area-System.Linq.Expressions
Sorry, something went wrong.
cc @fanyang-mono fyi
ivanpovazan
No branches or pull requests
As discussed in: #90519 both:
DynamicDelegateLightup
runtime/src/libraries/System.Linq.Expressions/src/System/Dynamic/Utils/DelegateHelpers.cs
Line 20 in b0d4502
ForceAllowDynamicCodeLightup
runtime/src/libraries/System.Linq.Expressions/src/System/Dynamic/Utils/DelegateHelpers.cs
Line 31 in b0d4502
should utilize
UnsafeAccessorAttribute
to simplify the logic of accessing the internal static methods inDelegate
andAssemblyBuilder
respectfully.This would additionally remove the need for
Internal.Runtime.Augments.DynamicDelegateAugments
forwarder and simplify the code in general.For reference: similar refactoring has been done in: #89706
The text was updated successfully, but these errors were encountered: