Skip to content

rustdoc: Fix some more assorted broken links #14592

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 7 commits into from
Jun 5, 2014

Conversation

alexcrichton
Copy link
Member

These are a few assorted fixes for some issues I found this morning (details in the commits).

If the type associated with the impl is `pub use`'d or referenced twice in a
downstream crate, the impl will attempt to be inlined twice.

Closes rust-lang#14584
@alexcrichton
Copy link
Member Author

Pushed a fix for #14594 as well

@alexcrichton alexcrichton reopened this Jun 2, 2014
use option::Option;

#[doc(hidden)]
Copy link
Member

Choose a reason for hiding this comment

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

Why are these hidden but Alignment etc not hidden?

Copy link
Member Author

Choose a reason for hiding this comment

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

Alignment is publicly available in the Formatter struct, but all these other bits are just internals of the fmt bytecode essentially. They all need to be public to be constructed anywhere.

Inlining caused implementors to show up multiple times.

cc rust-lang#14584
Apparently relying on provided_source in ty::Method is unreliable!

Closes rust-lang#14594
The logical location for the documentation of a primitive is in the module that
declared it was a module for that primitive.
@alexcrichton
Copy link
Member Author

Added a commit to put primitives in the right place in the documentation

r?

bors added a commit that referenced this pull request Jun 5, 2014
These are a few assorted fixes for some issues I found this morning (details in the commits).
@bors bors closed this Jun 5, 2014
@bors bors merged commit 1827241 into rust-lang:master Jun 5, 2014
@alexcrichton alexcrichton deleted the rustdoc-links branch June 5, 2014 06:59
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.

4 participants