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.

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

New lint: exhaustive tuple struct changed to plain struct #242

Closed
Tracked by #5
obi1kenobi opened this issue Dec 20, 2022 · 0 comments · Fixed by #249
Closed
Tracked by #5

New lint: exhaustive tuple struct changed to plain struct #242

obi1kenobi opened this issue Dec 20, 2022 · 0 comments · Fixed by #249
Assignees
Labels
A-lint Area: new or existing lint C-enhancement Category: raise the bar on expectations

Comments

@obi1kenobi
Copy link
Owner

obi1kenobi commented Dec 20, 2022

This is breaking even if the fields have the same names, because tuple struct patterns won't work.

Source: Rust for Rustaceans, Chapter 3, "Type Modifications", page 51

@obi1kenobi obi1kenobi changed the title exhaustive tuple struct to plain struct (even with fields with the same field names) is breaking because tuple struct patterns won't work (Rust for Rustaceans, Chapter 3, "Type Modifications", page 51) New lint: exhaustive tuple struct changed to plain struct Dec 20, 2022
@obi1kenobi obi1kenobi added A-lint Area: new or existing lint C-enhancement Category: raise the bar on expectations labels Dec 20, 2022
@staniewzki staniewzki self-assigned this Dec 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-lint Area: new or existing lint C-enhancement Category: raise the bar on expectations
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants