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

appreciative too_large_for_stack in useless vec! #5907

Merged
merged 1 commit into from
Aug 16, 2020

Conversation

wiomoc
Copy link
Contributor

@wiomoc wiomoc commented Aug 14, 2020

Fixes: #5847
changelog: Add too_large_for_stack configuration option for USELESS_VEC

@rust-highfive
Copy link

r? @phansch

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

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Aug 14, 2020
@wiomoc wiomoc force-pushed the feature/useless-vec-max-size branch from 92d9c45 to 8514b84 Compare August 14, 2020 12:29
@flip1995
Copy link
Member

@bors r+

Thanks!

@bors
Copy link
Contributor

bors commented Aug 16, 2020

📌 Commit 8514b84 has been approved by flip1995

@bors
Copy link
Contributor

bors commented Aug 16, 2020

⌛ Testing commit 8514b84 with merge 78857cc...

@bors
Copy link
Contributor

bors commented Aug 16, 2020

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: flip1995
Pushing 78857cc to master...

@bors bors merged commit 78857cc into rust-lang:master Aug 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties
Projects
None yet
Development

Successfully merging this pull request may close these issues.

useless_vec triggered by vectors that are too large to fit on the stack
5 participants