Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Box docs: no allocation is done for ZSTs. #40379

Merged
merged 1 commit into from
Mar 11, 2017
Merged

Conversation

clarfonthey
Copy link
Contributor

Updated to add a small bit saying that ZSTs don't actually allocate on Box::new.

@rust-highfive
Copy link
Collaborator

r? @brson

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

@brson
Copy link
Contributor

brson commented Mar 9, 2017

@bors r+ rollup

@bors
Copy link
Contributor

bors commented Mar 9, 2017

📌 Commit edf5dc6 has been approved by brson

arielb1 pushed a commit to arielb1/rust that referenced this pull request Mar 9, 2017
Box docs: no allocation is done for ZSTs.

Updated to add a small bit saying that ZSTs don't actually allocate on `Box::new`.
alexcrichton added a commit to alexcrichton/rust that referenced this pull request Mar 10, 2017
Box docs: no allocation is done for ZSTs.

Updated to add a small bit saying that ZSTs don't actually allocate on `Box::new`.
alexcrichton pushed a commit to arielb1/rust that referenced this pull request Mar 10, 2017
Box docs: no allocation is done for ZSTs.

Updated to add a small bit saying that ZSTs don't actually allocate on `Box::new`.
alexcrichton added a commit to alexcrichton/rust that referenced this pull request Mar 10, 2017
Box docs: no allocation is done for ZSTs.

Updated to add a small bit saying that ZSTs don't actually allocate on `Box::new`.
arielb1 pushed a commit to arielb1/rust that referenced this pull request Mar 10, 2017
Box docs: no allocation is done for ZSTs.

Updated to add a small bit saying that ZSTs don't actually allocate on `Box::new`.
arielb1 pushed a commit to arielb1/rust that referenced this pull request Mar 10, 2017
Box docs: no allocation is done for ZSTs.

Updated to add a small bit saying that ZSTs don't actually allocate on `Box::new`.
alexcrichton added a commit to alexcrichton/rust that referenced this pull request Mar 11, 2017
Box docs: no allocation is done for ZSTs.

Updated to add a small bit saying that ZSTs don't actually allocate on `Box::new`.
alexcrichton added a commit to alexcrichton/rust that referenced this pull request Mar 11, 2017
Box docs: no allocation is done for ZSTs.

Updated to add a small bit saying that ZSTs don't actually allocate on `Box::new`.
@bors bors merged commit edf5dc6 into rust-lang:master Mar 11, 2017
@clarfonthey clarfonthey deleted the box_docs branch March 11, 2017 19:22
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants