Skip to content
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

Tutorial: Added note about type parameters in ifaces. #2919

Merged
merged 2 commits into from
Jul 20, 2012
Merged

Tutorial: Added note about type parameters in ifaces. #2919

merged 2 commits into from
Jul 20, 2012

Conversation

Havvy
Copy link
Contributor

@Havvy Havvy commented Jul 14, 2012

Specficially, the type parameters should be left off of the function
signatures in both the iface and the impl if they are already in the
iface declaration.

Specficially, the type parameters should be left off of the function
signatures in both the iface and the impl if they are already in the
iface declaration.
@Havvy
Copy link
Contributor Author

Havvy commented Jul 14, 2012

The newer commit should be: use of the self type in interfaces.

@catamorphism catamorphism merged commit 2d23669 into rust-lang:incoming Jul 20, 2012
@catamorphism
Copy link
Contributor

I more or less rewrote the entire patch, but thanks for calling attention to these areas :-)

@Havvy
Copy link
Contributor Author

Havvy commented Jul 20, 2012

Given the amount of time that passed between the pull request and the merge, and the change to use traits and my lack of full understanding of what is going on, that is fully understandable.

RalfJung pushed a commit to RalfJung/rust that referenced this pull request Jun 29, 2023
Revert error in doc comment

Also punctuation
celinval pushed a commit to celinval/rust-dev that referenced this pull request Jun 4, 2024
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.

2 participants