Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

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

Jump to bottom

[17.2] Allow cached delegate for method groups #28942

Closed
Tracked by #30071
BillWagner opened this issue Apr 7, 2022 · 0 comments · Fixed by #28952
Closed
Tracked by #30071

[17.2] Allow cached delegate for method groups #28942

BillWagner opened this issue Apr 7, 2022 · 0 comments · Fixed by #28952
Assignees
@BillWagner
Labels
🏁 Release: .NET 7 Work items for the .NET 7 release doc-idea Indicates issues that are suggestions for new topics [org][type][category] Pri1 High priority, do before Pri2 and Pri3 resolved-by-customer Indicates issues where the customer no longer needs any help. [org][resolution]

Comments

@BillWagner
Copy link
Member

This has a small scope, and the spec is already updated. C# compilers are permitted to cache the delegate instance for a static method. See dotnet/roslyn#5835. This has minimal semantic impact on developers.
@PRMerger18 PRMerger18 added the Pri3 label Apr 7, 2022
@dotnet-bot dotnet-bot added the ⌚ Not Triaged Not triaged label Apr 7, 2022
@BillWagner BillWagner added doc-idea Indicates issues that are suggestions for new topics [org][type][category] Pri1 High priority, do before Pri2 and Pri3 🏁 Release: .NET 7 Work items for the .NET 7 release and removed Pri3 ⌚ Not Triaged Not triaged labels Apr 7, 2022
BillWagner added a commit to BillWagner/docs that referenced this issue Apr 8, 2022
@BillWagner
Document new method group conversion
66379bd 
Fixes dotnet#28942

Beginning in C# 11, the compiler is allowed to cache the delegate created from a method group conversion. The compiler now does this.
@BillWagner BillWagner mentioned this issue Apr 8, 2022
Merged
BillWagner added a commit that referenced this issue Apr 13, 2022
@BillWagner
Document new method group conversion (#28952)
8935584 
Fixes #28942

Beginning in C# 11, the compiler is allowed to cache the delegate created from a method group conversion. The compiler now does this.
@dotnet-bot dotnet-bot added the resolved-by-customer Indicates issues where the customer no longer needs any help. [org][resolution] label Apr 13, 2022
@BillWagner BillWagner self-assigned this Jul 1, 2022
@BillWagner BillWagner mentioned this issue Jul 1, 2022
Closed
21 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@BillWagner BillWagner

Labels
🏁 Release: .NET 7 Work items for the .NET 7 release doc-idea Indicates issues that are suggestions for new topics [org][type][category] Pri1 High priority, do before Pri2 and Pri3 resolved-by-customer Indicates issues where the customer no longer needs any help. [org][resolution]
Projects
No open projects
dotnet/docs: .NET 7 Release
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Document new method group conversion BillWagner/docs
3 participants
@BillWagner @dotnet-bot @PRMerger18