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

Rustdoc/linkage fixes #10003

Merged
merged 3 commits into from
Oct 22, 2013
Merged

Rustdoc/linkage fixes #10003

merged 3 commits into from
Oct 22, 2013

Conversation

alexcrichton
Copy link
Member

Commits have all the fun details.

When re-exporting a trait/structure/enum, then we need to propagate the
reachability of the type through the methods that are defined on it.

Closes rust-lang#9906
Closes rust-lang#9968
This does not work for cross-crate implementations of traits. Cross-crate
implementations are a separate issue that should be addressed separately.
Basically when an implementation of an external trait is detected, the trait
would have to be loaded at that time (or possibly sooner...). Rustdoc currently
doesn't have the proper infrastructure for adding this.

Closes rust-lang#9985
cc rust-lang#9999
bors added a commit that referenced this pull request Oct 22, 2013
@bors bors closed this Oct 22, 2013
@bors bors merged commit 41f6e97 into rust-lang:master Oct 22, 2013
flip1995 pushed a commit to flip1995/rust that referenced this pull request Dec 29, 2022
…_suggestion, r=Jarcho

fix incorrect suggestion in `suboptimal_flops`

fixes rust-lang#10003

There was an error when trying to negate an expression like `x - 1.0`. We used to format it as `-x - 1.0` whereas a proper negation would be `-(x - 1.0)`.

Therefore, we add parentheses around the expression when it is `ExprKind::Binary`.

We also add parentheses around multiply and divide expressions, even though this is not strictly necessary.

changelog: [`suboptimal_flops`]: fix incorrect suggestion caused by an incorrect negation of floating point expressions.
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