-
Notifications
You must be signed in to change notification settings - Fork 4k
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.
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
Remove support for inferring the return type of a local function. #7916
Conversation
// Specifically, the Interlocked.CompareExchange above succeeds, and will be seen by | ||
// other threads, before the diagnostics have been recorded in this symbol, below. | ||
// We can resolve this with a lock around this method and any other methods that | ||
// manipulate _diagnostics, directly or indirectly. | ||
AddDiagnostics(diagnostics.ToReadOnlyAndFree()); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Consider logging a bug for TODO.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done (#7938)
LGTM |
@dotnet-bot retest prtest/win/dbg/unit32 please |
The TODO should include a link to the bug. Otherwise LGTM |
I'm not sure about links from the compiler code to github. That would be a new precedent. On the other hand, it would make it easier to know which TODOs have already been made into issues. We're not likely to edit existing TODOs as we file issues for them, so I'd prefer to keep it this way. |
Remove support for inferring the return type of a local function.
Support for using `var` in a local function return type was already removed in PR dotnet#7916. This PR adds a test to verify that. Closes dotnet#10392.
Support for using `var` in a local function return type was already removed in PR dotnet#7916. This PR adds a test to verify that. Closes dotnet#10392.
Fixes #7832
@dotnet/roslyn-compiler Please review.