Skip to content

Add E0191, E0220 and E0232 error explanation #26879

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

Merged
merged 3 commits into from
Jul 9, 2015

Conversation

GuillaumeGomez
Copy link
Member

Part of #24407.
r? @Manishearth

@rust-highfive
Copy link
Contributor

r? @brson

(rust_highfive has picked a reviewer for you, use r? to override)

@@ -1550,6 +1550,30 @@ impl Foo for Bar {
```
"##,

E0191: r##"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You may want to make the reason clearer (trait objects need to have all associated types specified).

@GuillaumeGomez GuillaumeGomez changed the title Add E0191 error explanation Add E0191 and E0220 error explanation Jul 8, 2015
@GuillaumeGomez
Copy link
Member Author

@arielb1: Updated.

@GuillaumeGomez GuillaumeGomez changed the title Add E0191 and E0220 error explanation Add E0191, E0220 and E0232 error explanation Jul 8, 2015
// the trait `Trait`) must be specified
```

Please verify you specified all associated types of the trait or that you
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

*and

@arielb1
Copy link
Contributor

arielb1 commented Jul 8, 2015

r=me modulo grammar errors

@GuillaumeGomez
Copy link
Member Author

Squashed.

@arielb1
Copy link
Contributor

arielb1 commented Jul 8, 2015

@bors: r+ 6f01aa0

bors added a commit that referenced this pull request Jul 9, 2015
@bors
Copy link
Collaborator

bors commented Jul 9, 2015

⌛ Testing commit 6f01aa0 with merge 16f64c3...

@bors bors merged commit 6f01aa0 into rust-lang:master Jul 9, 2015
@GuillaumeGomez GuillaumeGomez deleted the patch-1 branch July 9, 2015 08:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants