Skip to content

Clearer error message for external trait and type #9682

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

skade
Copy link
Contributor

@skade skade commented Oct 2, 2013

The old error message implied that external traits could never
be implemented locally.

@skade
Copy link
Contributor Author

skade commented Oct 3, 2013

I considered it. neither and nor are quite a different level of english, though. Given that I know quite a few people that would struggle on that construction on first glance, I would opt for the easier language.

@alexcrichton
Copy link
Member

Could you squash these two commits into one? Also feel free to comment on the pull request when you update it because sadly github doesn't notify anyone on a force-push.

The old error message implied that external traits could never
be implemented locally.
@skade
Copy link
Contributor Author

skade commented Oct 6, 2013

@alexcrichton Sure, done.

bors added a commit that referenced this pull request Oct 6, 2013
…ype-and-trait, r=alexcrichton

The old error message implied that external traits could never
be implemented locally.
@bors bors closed this Oct 6, 2013
@bors bors merged commit 8154d23 into rust-lang:master Oct 6, 2013
@skade skade deleted the clearer-error-message-for-external-type-and-trait branch October 8, 2013 15:41
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.

3 participants