Add E0102, E0412, E0413, E0415, E0416, E0419, E0423 and E0435 error explanation #27378
Conversation
|
I'm not sure at all this is enough to fully explain this error. Waiting for opinions. |
GuillaumeGomez
changed the title
| // unit-like struct in scope | ||
| ``` | ||
|
|
||
| To fix this error, you have to change the declaration's name or the |
There was a problem hiding this comment.
you have to change the name of the declaration of shadowed object
|
Leave out 424, the original error is wrong (it applies to both structs and struct variants, in positions other than function positions) |
|
|
||
|
|
||
| To fix this error, you have to change the name of the declaration of | ||
| shadowed object. Please also verify that neither name was misspelled. |
There was a problem hiding this comment.
s/shadowed object/the shadowed object/
GuillaumeGomez
changed the title
GuillaumeGomez
changed the title
GuillaumeGomez
changed the title
|
Since #27491 has been merged. We can move forward on this PR. |
GuillaumeGomez
changed the title
GuillaumeGomez
changed the title
GuillaumeGomez
changed the title
| @@ -1118,6 +1118,36 @@ fn main() { | |||
| ``` | |||
| "##, | |||
|
|
|||
| E0102: r##" | |||
| You hit this error because the compiler the compiler lacks information | |||
| "##, | ||
|
|
||
| E0423: r##" | ||
| A `struct` variant name was used like a function name. Example of |
There was a problem hiding this comment.
struct “variant”? Interestingly it was called so by yourself in c5f7c19. Stable and beta emit a better error message in this case.
There was a problem hiding this comment.
So should I change it or not ?
There was a problem hiding this comment.
I’m fine with merging this as it is currently and changing the explanation in the PR which updates the error message.
| @@ -397,6 +397,102 @@ impl Bar { | |||
| ``` | |||
| "##, | |||
|
|
|||
| E0412: r##" | |||
| An undeclared type name was used. Example of erroneous codes: | |||
There was a problem hiding this comment.
May want to say something about generics, e.g. fn foo(x: T) {} needing to be fn foo<T>(x: T) {}.
|
@bors r+ |
|
📌 Commit c3d147e has been approved by |
|
💔 Test failed - auto-linux-32-opt |
|
@bors retry |
|
⚡ Previous build results for auto-win-msvc-32-opt, auto-win-msvc-64-opt are reusable. Rebuilding only auto-linux-32-nopt-t, auto-linux-32-opt, auto-linux-64-nopt-t, auto-linux-64-opt, auto-linux-64-x-android-t, auto-mac-32-opt, auto-mac-64-nopt-t, auto-mac-64-opt, auto-win-gnu-32-nopt-t, auto-win-gnu-32-opt, auto-win-gnu-64-nopt-t, auto-win-gnu-64-opt... |
Part of #24407.
r? @Manishearth