-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
Resolve overload ambiguities from lambda expression inference change #58953
Conversation
Tagging subscribers to this area: @tarekgh, @dotnet/area-system-linq-parallel Issue DetailsResolve overload ambiguities resulting from C#10 compiler changes to inference with lambda expressions. See #58873 (comment) cc @stephentoub
|
cc @ericstj, @ViktorHofer |
I would prefer we not take this, and we first see what we can do in the language to make these retain their original meaning. |
Is there a spec or description of how the ambiguities are supposed to be resolved? |
We can revert this change later if the C# compiler is updated to no longer report these cases as ambiguous. In the meantime, this change, or something similar, is needed to allow moving to the latest compiler. |
The test changes themselves look fine. |
agree with @cston, halting codeflow is bad as it allows changes to accumulate. I've been waiting on arcade flow since Monday (due to an assortment of issues), let's please get it unblocked. File an issue, with a nice repro and use this as a test case for that fix. |
Resolve overload ambiguities resulting from C#10 compiler changes to inference with lambda expressions.
See #58873 (comment)
cc @stephentoub