-
Notifications
You must be signed in to change notification settings - Fork 13.2k
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
Confusing errors with missing let
in if let
#77238
Labels
A-diagnostics
Area: Messages for errors, warnings, and lints
A-type-system
Area: Type system
D-confusing
Diagnostics: Confusing error or lint that should be reworked.
D-newcomer-roadblock
Diagnostics: Confusing error or lint; hard to understand for new users.
T-compiler
Relevant to the compiler team, which will review and decide on the PR/issue.
Comments
Dylan-DPC-zz
pushed a commit
to Dylan-DPC-zz/rust
that referenced
this issue
Oct 26, 2020
…crum Tweak `if let` suggestion to be more liberal with suggestion and to not ICE Fix rust-lang#77218. Fix rust-lang#77238.
Dylan-DPC-zz
pushed a commit
to Dylan-DPC-zz/rust
that referenced
this issue
Oct 26, 2020
…crum Tweak `if let` suggestion to be more liberal with suggestion and to not ICE Fix rust-lang#77218. Fix rust-lang#77238.
bors
added a commit
to rust-lang-ci/rust
that referenced
this issue
Oct 26, 2020
Tweak `if let` suggestion to be more liberal with suggestion and to not ICE Fix rust-lang#77218. Fix rust-lang#77238.
Mark-Simulacrum
pushed a commit
to Mark-Simulacrum/rust
that referenced
this issue
Nov 6, 2020
Reopening as per #77283 (comment) |
This is fixed in beta now:
It is still quite verbose (too many errors), but we lead people in the right direction. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
A-diagnostics
Area: Messages for errors, warnings, and lints
A-type-system
Area: Type system
D-confusing
Diagnostics: Confusing error or lint that should be reworked.
D-newcomer-roadblock
Diagnostics: Confusing error or lint; hard to understand for new users.
T-compiler
Relevant to the compiler team, which will review and decide on the PR/issue.
I thought this code would provide one of the nice suggestions that @estebank implemented in #75931, but it doesn't. (By the way, making
cache
a reference toCache
will give an ICE because of #77218.)(Playground)
Errors:
The text was updated successfully, but these errors were encountered: