Skip to content

Conversation

lukas-code
Copy link
Member

The docs for vec-box-size-threshold say "The size of the boxed type in bytes, where boxing in a Vec is allowed", but previously a boxed type of exactly that size wasn't allowed in Vec.

changelog: [vec_box]: Fix off-by-one error for vec-box-size-threshold configuration.

@rust-highfive
Copy link

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @Jarcho (or someone else) soon.

Please see the contribution instructions for more information.

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Nov 14, 2022
@Jarcho
Copy link
Contributor

Jarcho commented Nov 14, 2022

Thank you. @bors r+

@bors
Copy link
Contributor

bors commented Nov 14, 2022

📌 Commit b8357ff has been approved by Jarcho

It is now in the queue for this repository.

@bors
Copy link
Contributor

bors commented Nov 14, 2022

⌛ Testing commit b8357ff with merge 3c7f74d...

@bors
Copy link
Contributor

bors commented Nov 14, 2022

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: Jarcho
Pushing 3c7f74d to master...

@bors bors merged commit 3c7f74d into rust-lang:master Nov 14, 2022
@lukas-code lukas-code deleted the vec-box branch November 14, 2022 16:10
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.

4 participants