Skip to content

Conversation

apasel422 and others added 22 commits October 13, 2015 09:44
Before this commit, the first "A Rust library" code sample produced
the following compilation warning:
```
test.rs:7:22: 7:36 warning: unnecessary parentheses around `for` head
expression, #[warn(unused_parens)] on by default
test.rs:7             for _ in (0..5_000_000) {
```

This commit just removes the parens around the range 0..5_000_000.
r? @nrc

I was wondering, what is the standard for pattern matching in presence of guards? I liked more the original alignment of `distributions/gamma.rs` line 91 than the new one.
…ning, r=alexcrichton

Before this commit, the first "A Rust library" code sample produced
the following compilation warning:
```
test.rs:7:22: 7:36 warning: unnecessary parentheses around `for` head
expression, #[warn(unused_parens)] on by default
test.rs:7             for _ in (0..5_000_000) {
```

This commit just removes the parens around the range 0..5_000_000 thereby removing the compilation warning.
…bnik

Regarding [rust-lang#29063 _[Docs] Terminology inconsistency between 'iterator adapters' and 'iterator adaptors'_](rust-lang#29063) :

This PR replaces 'iterator adapters' appearances (in TRPL book) to 'iterator adaptors', thus embracing the terminology used along the API docs and achieving consistency between both sources.
@steveklabnik
Copy link
Contributor Author

@bors: r+ p=1

@bors
Copy link
Collaborator

bors commented Oct 15, 2015

📌 Commit 46d8a76 has been approved by steveklabnik

@rust-highfive
Copy link
Contributor

r? @alexcrichton

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

@bors
Copy link
Collaborator

bors commented Oct 15, 2015

⌛ Testing commit 46d8a76 with merge eac6986...

@bors
Copy link
Collaborator

bors commented Oct 15, 2015

💔 Test failed - auto-linux-64-nopt-t

@Centril Centril added the rollup A PR which is a rollup label Oct 24, 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.