Skip to content

Guide: Closures: minor wording fixes #18340

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

Merged
merged 2 commits into from
Oct 29, 2014

Conversation

chastell
Copy link
Contributor

Some minor wording fixes to the Closures chapter; my brain tripped a few times when reading it, so I tried to come up with something a bit smoother. I’m not a native speaker, so please do review this critically.

@rust-highfive
Copy link
Contributor

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @nikomatsakis (or someone else) soon.

incredibly powerful. Next, let's look at one of those things: iterators.
strange at first, but once you're used to them, you'll miss them
in other languages. Passing functions to other functions is
incredibly powerful; let's look at one of such situations: iterators.
Copy link
Contributor

Choose a reason for hiding this comment

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

This last sentence sounds strange ;-) Suggestion: ... is incredibly powerful as you will see in the following chapter about iterators.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good call; fixed as per your suggestion. :)

bors added a commit that referenced this pull request Oct 29, 2014
…bnik

Some minor wording fixes to the Closures chapter; my brain tripped a few times when reading it, so I tried to come up with something a bit smoother. I’m not a native speaker, so please do review this critically.
@bors bors closed this Oct 29, 2014
@bors bors merged commit 16bae69 into rust-lang:master Oct 29, 2014
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.

6 participants