Skip to content

Clarify text on pattern exhaustiveness #939

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
Closed
BillWagner opened this issue Sep 16, 2023 · 1 comment
Closed

Clarify text on pattern exhaustiveness #939

BillWagner opened this issue Sep 16, 2023 · 1 comment
Labels
type: clarity While not technically incorrect, the Standard is potentially confusing
Milestone
Pre-C# 8.0

Comments

@BillWagner
Copy link
Member

See #757 (comment)

In 11.4, a bit of wordsmithing could improve the text.

Deferred until after C# 7
@BillWagner BillWagner mentioned this issue Sep 16, 2023
Merged
@jskeet jskeet added this to the C# 8.0 milestone Sep 20, 2023
@jskeet jskeet added the type: clarity While not technically incorrect, the Standard is potentially confusing label Sep 20, 2023
@jskeet jskeet modified the milestones: C# 8.0, Pre-C# 8.0 Sep 21, 2023
@Nigel-Ecma Nigel-Ecma mentioned this issue Oct 3, 2023
Merged
@jskeet
Copy link
Contributor

jskeet commented Oct 4, 2023

Fixed in #963.

@jskeet jskeet closed this as completed Oct 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
type: clarity While not technically incorrect, the Standard is potentially confusing
Projects
None yet
Milestone
Pre-C# 8.0
Development

No branches or pull requests
2 participants
@jskeet @BillWagner