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

Improve doc for std::mem::drop #26664

Merged
merged 1 commit into from
Jun 30, 2015
Merged

Improve doc for std::mem::drop #26664

merged 1 commit into from
Jun 30, 2015

Conversation

steveklabnik
Copy link
Member

This was pretty misleading, so let's improve.

Fixes #26571

@rust-highfive
Copy link
Collaborator

r? @nikomatsakis

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

@alexcrichton
Copy link
Member

r=me, but looks like a tidy error (I'd also be fine wrapping to 80 chars)

@steveklabnik
Copy link
Member Author

man we really need to get this 80/100 split fixed

@steveklabnik
Copy link
Member Author

@bors: r=alexcrichton rollup

This was pretty misleading, so let's improve.

Fixes rust-lang#26571
@bors
Copy link
Contributor

bors commented Jun 29, 2015

📌 Commit f758baa has been approved by alexcrichton

@bors
Copy link
Contributor

bors commented Jun 30, 2015

⌛ Testing commit f758baa with merge 0b70378...

bors added a commit that referenced this pull request Jun 30, 2015
This was pretty misleading, so let's improve.

Fixes #26571
@bors bors merged commit f758baa into rust-lang:master Jun 30, 2015
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