Skip to content
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

Make note of performance implications of Read #28762

Merged
merged 1 commit into from
Sep 30, 2015

Conversation

steveklabnik
Copy link
Member

Fixes #28073

@rust-highfive
Copy link
Collaborator

r? @nikomatsakis

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

@nikomatsakis
Copy link
Contributor

@bors r+ rollup

though I might add a nit to say something like "will likely be faster" :)

@bors
Copy link
Contributor

bors commented Sep 30, 2015

📌 Commit 367f46d has been approved by nikomatsakis

@steveklabnik
Copy link
Member Author

steveklabnik commented Sep 30, 2015 via email

steveklabnik added a commit to steveklabnik/rust that referenced this pull request Sep 30, 2015
bors added a commit that referenced this pull request Sep 30, 2015
@bors bors merged commit 367f46d into rust-lang:master Sep 30, 2015
@steveklabnik steveklabnik deleted the gh28073 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.

4 participants