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

Deprecate {Box,Rc,Arc}::into_raw_non_null #71168

Merged
merged 4 commits into from
Apr 25, 2020

Conversation

SimonSapin
Copy link
Contributor

Per ongoing FCP at #47336 (comment)
See also #47336 (comment)

@rust-highfive
Copy link
Collaborator

r? @kennytm

(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 Apr 15, 2020
src/liballoc/boxed.rs Outdated Show resolved Hide resolved
src/liballoc/boxed.rs Outdated Show resolved Hide resolved
src/liballoc/sync.rs Outdated Show resolved Hide resolved
Co-Authored-By: Ralf Jung <post@ralfj.de>
src/liballoc/boxed.rs Outdated Show resolved Hide resolved
src/liballoc/boxed.rs Outdated Show resolved Hide resolved
… instead of the other way around.
Copy link
Member

@RalfJung RalfJung left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All looking good from my side, thanks. :)

@crlf0710 crlf0710 added the T-libs-api Relevant to the library API team, which will review and decide on the PR/issue. label Apr 24, 2020
@RalfJung RalfJung mentioned this pull request Apr 25, 2020
@Amanieu
Copy link
Member

Amanieu commented Apr 25, 2020

@bors r+

@bors
Copy link
Contributor

bors commented Apr 25, 2020

📌 Commit 7709d20 has been approved by Amanieu

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Apr 25, 2020
bors added a commit to rust-lang-ci/rust that referenced this pull request Apr 25, 2020
Rollup of 7 pull requests

Successful merges:

 - rust-lang#69041 (proc_macro: Stabilize `Span::resolved_at` and `Span::located_at`)
 - rust-lang#69813 (Implement BitOr and BitOrAssign for the NonZero integer types)
 - rust-lang#70712 (stabilize BTreeMap::remove_entry)
 - rust-lang#71168 (Deprecate `{Box,Rc,Arc}::into_raw_non_null`)
 - rust-lang#71544 (Replace filter_map().next() calls with find_map())
 - rust-lang#71545 (Fix comment in docstring example for Error::kind)
 - rust-lang#71548 (Add missing Send and Sync impls for linked list Cursor and CursorMut.)

Failed merges:

r? @ghost
@bors bors merged commit 939c932 into rust-lang:master Apr 25, 2020
yvt added a commit to yvt/Stella2 that referenced this pull request May 4, 2020
`Box::into_raw_non_null` was recently deprecated by
<rust-lang/rust#71168> and will probably be
removed after some time.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-libs-api Relevant to the library API team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants