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

Rollup of 9 pull requests #25958

Merged
merged 18 commits into from
Jun 2, 2015
Merged

Rollup of 9 pull requests #25958

merged 18 commits into from
Jun 2, 2015

Conversation

jooert and others added 18 commits May 27, 2015 11:28
Method where clauses are put indented on a new line, do the same tweak
to free functions, which makes it cleaner and easier to read.
As of rust-lang#22980 only `cfg(debug_assertions)` is used in the
standard library and rustc code.
This is based on the documented behavior of cmath::{fmin,fmax}.  It is also
tested by existing unit tests in this module.
rustdoc: Tweak css of function where clauses

Method where clauses are put indented on a new line, do the same tweak
to free functions, which makes it cleaner and easier to read.
As of rust-lang#22980 only `cfg(debug_assertions)` is used in the
standard library and rustc code.
This is based on the documented behavior of cmath::{fmin,fmax}.  It is also tested by existing unit tests in this module.
@rust-highfive
Copy link
Collaborator

r? @nikomatsakis

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

@Manishearth
Copy link
Member Author

@bors: r+ p=20

@bors
Copy link
Contributor

bors commented Jun 2, 2015

📌 Commit 6e97b16 has been approved by Manishearth

@bors
Copy link
Contributor

bors commented Jun 2, 2015

⌛ Testing commit 6e97b16 with merge 48e9ef6...

bors added a commit that referenced this pull request Jun 2, 2015
@bors bors merged commit 6e97b16 into rust-lang:master Jun 2, 2015
@Centril Centril added the rollup A PR which is a rollup label Oct 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
rollup A PR which is a rollup
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants