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

Round generator sizes to a multiple of their alignment #63208

Merged
merged 1 commit into from Aug 3, 2019

Conversation

@tmandry
Copy link
Contributor

commented Aug 2, 2019

Fixes #62658.

r? @cramertj
cc @eddyb

{
let x = [0u8; 17];
noop().await;
println!("{:?}", x);

This comment has been minimized.

Copy link
@hellow554

hellow554 Aug 2, 2019

Contributor

Instead of println you could use either drop is define a fn dummy<T>(_: T) {} and use that

@cramertj

This comment has been minimized.

Copy link
Member

commented Aug 2, 2019

@bors r+

@bors

This comment has been minimized.

Copy link
Contributor

commented Aug 2, 2019

📌 Commit 14be088 has been approved by cramertj

Centril added a commit to Centril/rust that referenced this pull request Aug 2, 2019

Rollup merge of rust-lang#63208 - tmandry:issue-62658, r=cramertj
Round generator sizes to a multiple of their alignment

Fixes rust-lang#62658.

r? @cramertj
cc @eddyb
@Centril Centril referenced this pull request Aug 2, 2019

bors added a commit that referenced this pull request Aug 2, 2019

Auto merge of #63228 - Centril:rollup-x39p5ga, r=Centril
Rollup of 7 pull requests

Successful merges:

 - #63107 (Added support for armv7-unknown-linux-gnueabi/musleabi)
 - #63121 (On `format!()` arg count mismatch provide extra info)
 - #63196 (build_helper: try less confusing method names)
 - #63206 (remove unsupported test case)
 - #63208 (Round generator sizes to a multiple of their alignment)
 - #63212 (Pretty print attributes in `print_arg`)
 - #63215 (Clarify semantics of mem::zeroed)

Failed merges:

r? @ghost

@bors bors merged commit 14be088 into rust-lang:master Aug 3, 2019

4 checks passed

pr Build #20190802.5 succeeded
Details
pr (Linux mingw-check) Linux mingw-check succeeded
Details
pr (Linux x86_64-gnu-llvm-6.0) Linux x86_64-gnu-llvm-6.0 succeeded
Details
pr (LinuxTools) LinuxTools succeeded
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants
You can’t perform that action at this time.