Skip to content

Conversation

steveklabnik
Copy link
Contributor

  1. mention them in the function chapter
  2. mention their coercion to closures in the closures chapter

Fixes #26746

1. mention them in the function chapter
2. mention their coercion to closures in the closures chapter

Fixes rust-lang#26746
@rust-highfive
Copy link
Contributor

r? @brson

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

@brson
Copy link
Contributor

brson commented Aug 5, 2015

@bors r+

@bors
Copy link
Collaborator

bors commented Aug 5, 2015

📌 Commit e4c229b has been approved by brson

@steveklabnik
Copy link
Contributor Author

@bors: rollup

steveklabnik added a commit to steveklabnik/rust that referenced this pull request Aug 5, 2015
1. mention them in the function chapter
2. mention their coercion to closures in the closures chapter

Fixes rust-lang#26746
bors added a commit that referenced this pull request Aug 5, 2015
In this example, we don’t strictly need the intermediate variable `f`,
the name of the function works just fine too:

```ignore
Copy link
Contributor

Choose a reason for hiding this comment

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

Should be rust,ignore

Copy link
Contributor Author

Choose a reason for hiding this comment

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

rust is assumed by default

Copy link
Contributor

Choose a reason for hiding this comment

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

@steveklabnik Only when using rustdoc, but not when generating the ebooks. I'd like to keep the Markdown sources as universal as possible. Don't worry about it, though. I'll send a PR later today to add rust where necessary.

@bors bors merged commit e4c229b into rust-lang:master Aug 5, 2015
@steveklabnik steveklabnik deleted the gh26746 branch June 19, 2016 20:32
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.

5 participants