-
Notifications
You must be signed in to change notification settings - Fork 13.3k
Add E0092, E0093, E0094, E0101, E0117, E0119, E0195, E0207 and E0211 error explanation #26651
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
Conversation
struct Foo; | ||
|
||
impl Trait for Foo { | ||
fn t<'a,'b>(x: &'a str, y: &'b str) { // error: lifetime parameters |
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.
maybe the comment could go to the next line (or get deindented on the second line) instead of wrapping and spanning 4 whole lines?
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 saw it like this in other error explanations. So I don't mind. I'll wait for other opinions before changing it, we'll see what people say.
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 prefer having it on the next line
@@ -1022,6 +1022,119 @@ type Foo<A> = Box<A>; // ok! | |||
``` | |||
"##, | |||
|
|||
E0092: r##" | |||
You tried to call an undefined atomic operation function. |
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.
"tried to declare"
@Manishearth: Updated! Do you see anything else ? |
} | ||
``` | ||
|
||
The 'b lifetime constraints for `t` implementation does not match the |
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.
- Backticks around
'b
- "The lifetime constraint
'b
for the implementation oft()
does not match the trait declaration" - replacing
t
withmethod
might be better
@bors r+ rollup |
📌 Commit 5aeecd2 has been approved by |
@bors r- |
Still missing #26651 (diff) |
fn foo(); // error: unrecognized intrinsic function: `foo` | ||
} | ||
|
||
fn main() { |
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.
Do you need this function to exist in order to reproduce the error?
Anything else to fix/change ? |
Squashed. |
@bors r+ rollup |
📌 Commit be38926 has been approved by |
Part of #24407.
cc @michaelsproul
r? @Manishearth