Skip to content

docs show different method definitions for core and std #17693

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.

Sign up for GitHub

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

Jump to bottom
Closed
lucidd opened this issue Oct 1, 2014 · 4 comments
Closed

docs show different method definitions for core and std #17693

lucidd opened this issue Oct 1, 2014 · 4 comments
Labels
T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue.

Comments

@lucidd
Copy link
Contributor

lucidd commented Oct 1, 2014

I noticed that the docs on Iterator differ depending on if you look a core::iter::Iterator or std::iter::Iterator. In the std version the declaration of the lifetime 'r in the methods is missing.
Im not sure if this is a bug or intended but its confusing if you look at the std version and wonder where lifetime 'r comes from.
@reem
Copy link
Contributor

reem commented Oct 1, 2014

@lucidd Your links are swapped.

@lucidd
Copy link
Contributor Author

lucidd commented Oct 1, 2014

@reem thanks changed it.

@steveklabnik
Copy link
Member

This is a bug and we have an open issue, or so I thought, but I can't find it.

@steveklabnik steveklabnik added the T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue. label Oct 1, 2014
@lucidd
Copy link
Contributor Author

lucidd commented Oct 1, 2014

@steveklabnik i think i found it #14462 so closing this as a duplicate

@lucidd lucidd closed this as completed Oct 1, 2014
lnicola pushed a commit to lnicola/rust that referenced this issue Jul 28, 2024
@bors
Auto merge of rust-lang#17693 - Veykril:astidmap, r=Veykril
5bded7c 
Remove Params and Fields from AstIdMap

These are too volatile, and the only reason for them seems to be for cfg diagnostics which does not carry the weight
RalfJung pushed a commit to RalfJung/rust that referenced this issue Aug 1, 2024
@bors
Auto merge of rust-lang#17693 - Veykril:astidmap, r=Veykril
a226dbf 
Remove Params and Fields from AstIdMap

These are too volatile, and the only reason for them seems to be for cfg diagnostics which does not carry the weight
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@steveklabnik @lucidd @reem