Skip to content

Fix duplicate methods in rustdoc #22765

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
Feb 27, 2015
Merged

Fix duplicate methods in rustdoc #22765

merged 3 commits into from
Feb 27, 2015

Conversation

sanxiyn
Copy link
Member

@sanxiyn sanxiyn commented Feb 24, 2015

rustdoc impl item did not include default methods for local crates, but did include them for external crates. This resulted in duplicate methods. Fix so that impl item does not include default methods for external crates.

Fix #22595.

@rust-highfive
Copy link
Contributor

r? @nikomatsakis

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

@nikomatsakis
Copy link
Contributor

can you make a test? there is some infrastructure for this in src/test/run-make/rustdoc-*/, but I haven't investigated too deeply.

@sanxiyn
Copy link
Member Author

sanxiyn commented Feb 24, 2015

I couldn't figure out how to test this easily. Maybe file an issue to test this?

@nikomatsakis
Copy link
Contributor

what makes it hard to test exactly?

@sanxiyn
Copy link
Member Author

sanxiyn commented Feb 24, 2015

htmldocck.py can test for one or more occurences, but can't test for exactly one occurence at the moment. I can implement that and then test this. I will try that tomorrow.

@sanxiyn
Copy link
Member Author

sanxiyn commented Feb 26, 2015

Added a test.

@alexcrichton
Copy link
Member

@bors: r+ 6958463

Nice!

bors added a commit that referenced this pull request Feb 27, 2015
rustdoc impl item did not include default methods for local crates, but did include them for external crates. This resulted in duplicate methods. Fix so that impl item does not include default methods for external crates.

Fix #22595.
@bors
Copy link
Collaborator

bors commented Feb 27, 2015

⌛ Testing commit 6958463 with merge e5cd653...

@bors
Copy link
Collaborator

bors commented Feb 27, 2015

@bors bors merged commit 6958463 into rust-lang:master Feb 27, 2015
@sanxiyn sanxiyn deleted the dedup-rustdoc branch February 27, 2015 05:40
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.

Rustdoc shows duplicate methods
5 participants