-
Notifications
You must be signed in to change notification settings - Fork 13.4k
Add error code flags #34401
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
Add error code flags #34401
Conversation
@@ -947,6 +945,8 @@ position that needs that trait. For example, when the following code is | |||
compiled: | |||
|
|||
```compile_fail | |||
#![feature(on_unimplemented)] | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why did these examples start requiring #![feature(on_unimplemented)]
but still not get error code annotations?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Because in order to use #[rustc_on_unimplemented]
, you need to enable the feature. However, the code throws E0230
, so I can't add E0272
. The same goes for E0273
and E0274
.
@bors r+ |
📌 Commit 8f987ab has been approved by |
Add error code flags r? @brson cc @steveklabnik cc @jonathandturner
// nothing here | ||
```compile_fail,E0269 | ||
fn abracada_FAIL() -> String { | ||
"this won't work".to_string(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this change intentional?
It seems the source code does not match the following explanation text anymore.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah damn, i didn't pay attention to the explanation. Want to fix it?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry, it don't have enough time/knowledge to fix this in short time frame.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll then. Thanks for notifying me!
r? @brson
cc @steveklabnik
cc @jonathandturner