Add E0417, E0424, E0425, E0426, E0430, E0431 and E0432 error explanations #27230
Conversation
|
r? @pnkfelix (rust_highfive has picked a reviewer for you, use r? to override) |
GuillaumeGomez
changed the title
| } | ||
| ``` | ||
|
|
||
| Compiler needs to know the pattern value at compile's time, which is |
There was a problem hiding this comment.
The compiler needs to know the value of the pattern at compile time,
| ``` | ||
|
|
||
| The compiler needs to know the value of the pattern at compile time, | ||
| which is not possible with a `static` variable. Please verify that the |
There was a problem hiding this comment.
nit: saying "is not possible" may be re-opening a can of worms by implicitly inviting back those who were arguing, back when we first split up the two forms, that the static / const distinction was unnecessary.
I would prefer something like:
The compiler needs to know the value of the pattern at compile time; compile-time patterns can defined via
constorenumitems. Please verify that the identifier is spelled correctly, and if so, use aconstinstead ofstaticto define it.
Example:
...
GuillaumeGomez
changed the title
GuillaumeGomez
changed the title
GuillaumeGomez
changed the title
GuillaumeGomez
changed the title
GuillaumeGomez
changed the title
| } | ||
| ``` | ||
|
|
||
| Please check if the method's argument list should have contained self, |
|
@bors r+ |
|
📌 Commit fc65f55 has been approved by |
Part of #24407.
cc @Manishearth