Skip to content

Fix two small issues in iterator docs#37963

Merged
bors merged 2 commits into
rust-lang:masterfrom
bluss:iterator-docs
Nov 26, 2016
Merged

Fix two small issues in iterator docs#37963
bors merged 2 commits into
rust-lang:masterfrom
bluss:iterator-docs

Conversation

@bluss

@bluss bluss commented Nov 23, 2016

Copy link
Copy Markdown
Contributor
  • collect() is a regular method, not an adaptor (does not return an Iterator). I just randomly picked filter as a third common adaptor to mention instead.
  • Fix example in Map's docs so that it uses the DoubleEndedIterator implementation

Make the example use DoubleEndedIterator for map, like it said it would.
@rust-highfive

Copy link
Copy Markdown
Contributor

r? @alexcrichton

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

@alexcrichton

Copy link
Copy Markdown
Member

@bors: r+

@bors

bors commented Nov 23, 2016

Copy link
Copy Markdown
Collaborator

📌 Commit 557369e has been approved by alexcrichton

@bluss

bluss commented Nov 24, 2016

Copy link
Copy Markdown
Contributor Author

@bors rollup

sanxiyn added a commit to sanxiyn/rust that referenced this pull request Nov 26, 2016
Fix two small issues in iterator docs

- `collect()` is a regular method, not an adaptor (does not return an Iterator). I just randomly picked `filter` as a third common adaptor to mention instead.
- Fix example in `Map`'s docs so that it uses the DoubleEndedIterator implementation
bors added a commit that referenced this pull request Nov 26, 2016
Rollup of 7 pull requests

- Successful merges: #37962, #37963, #37967, #37978, #37985, #38001, #38010
- Failed merges:
@bors bors merged commit 557369e into rust-lang:master Nov 26, 2016
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