Add breaking change item for forbidding default in pattern.#25800
Conversation
gafter
added
Area-Compilers
Documentation
Feature - Pattern Matching
| } | ||
| ``` | ||
|
|
||
| - https://github.com/dotnet/roslyn/issues/25450 In C# 7.3 we add new restrictions on the use of the `default` expression in preparation for the planned addition of further pattern-matching features in C# 8.0: |
There was a problem hiding this comment.
Was the change made in 15.7? I think it would be good to specify that version.
There was a problem hiding this comment.
just a thought: I think saying "In C# 7.3 we add new restrictions" is a little misleading because it makes it sound like this change is only going to affect you if you're using language version 7.3 or newer, but that's not the case as far as I know. It affects 7.1 as well, doesn't it?
There was a problem hiding this comment.
I think this might give someone a false sense of security thinking their code will still work with the new compiler until they switch to a newer language version
There was a problem hiding this comment.
@gafter I made an edit directly to you branch to remove the mention of language version 7.3. I hope that's ok. Thanks
|
test windows_debug_unit32_prtest please |
|
windows_release_unit64_prtest failed (details) FYI @dotnet/roslyn-infrastructure I'll re-run. |
|
test windows_release_unit64_prtest please |
|
test windows_debug_unit64_prtest please |
|
test windows_release_unit32_prtest please |
|
@gafter Should we call how this change in the 15.7 release notes since it seems a few people hit it already? |
3 participants
@jcouv Please review
/cc @dotnet/roslyn-compiler