Skip to content

Minor grammar/spelling fixes to the "Static and Dynamic Dispatch" doc #22625

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 1 commit into from
Feb 22, 2015

Conversation

inrustwetrust
Copy link
Contributor

As far as I could tell from discussions in other docs pull requests, American spelling is preferred for the docs?

@rust-highfive
Copy link
Contributor

r? @huonw

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

@inrustwetrust
Copy link
Contributor Author

r? @steveklabnik

@rust-highfive rust-highfive assigned steveklabnik and unassigned huonw Feb 21, 2015
@@ -84,7 +84,7 @@ inlining and hence usually higher performance. It also has some downsides:
causing code bloat due to many copies of the same function existing in the
binary, one for each type.

Furthermore, compilers aren’t perfect and may “optimise” code to become slower.
Furthermore, compilers aren’t perfect and may “optimize” code to become slower.
Copy link
Member

Choose a reason for hiding this comment

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

Nooooo, my article my spelling!

:p

(Thanks for updating these.)

@huonw
Copy link
Member

huonw commented Feb 21, 2015

@bors r+ 1b20 rollup

Manishearth added a commit to Manishearth/rust that referenced this pull request Feb 21, 2015
… r=huonw

 As far as I could tell from discussions in other docs pull requests, American spelling is preferred for the docs?
@huonw huonw merged commit 1b20767 into rust-lang:master Feb 22, 2015
@inrustwetrust inrustwetrust deleted the dispatch-doc-grammar branch February 22, 2015 10:29
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