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

Stabilize UnsafeCell::from_mut #131261

Merged
merged 1 commit into from
Nov 6, 2024
Merged

Conversation

clarfonthey
Copy link
Contributor

@clarfonthey clarfonthey commented Oct 4, 2024

Closes #111645.
FCP: #111645 (comment)

Note that because const_mut_refs and const_refs_to_cell was stabilized, it's okay to const-stabilize this method as well.

@rustbot
Copy link
Collaborator

rustbot commented Oct 4, 2024

r? @tgross35

rustbot has assigned @tgross35.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-libs Relevant to the library team, which will review and decide on the PR/issue. labels Oct 4, 2024
@tgross35
Copy link
Contributor

tgross35 commented Oct 4, 2024

This seems fine to me, but FCP did not include const stabilization so

r? libs-api

Cc @rust-lang/wg-const-eval

@rustbot rustbot added the T-libs-api Relevant to the library API team, which will review and decide on the PR/issue. label Oct 4, 2024
@rustbot rustbot assigned dtolnay and unassigned tgross35 Oct 4, 2024
@RalfJung
Copy link
Member

RalfJung commented Oct 4, 2024

No objections from the const-eval side.

But yeah the question is whether libs-api wants to retroactively add const to the FCP or start a new FCP.

@RalfJung RalfJung added the I-libs-api-nominated Nominated for discussion during a libs-api team meeting. label Oct 4, 2024
@dtolnay
Copy link
Member

dtolnay commented Oct 6, 2024

I am not unilaterally approving a const stabilization on top of the team FCP from #111645. But if there is quorum in the next libs-api meeting or enough 👍 from @rust-lang/libs-api, I think this PR can be merged without a whole new FCP.

@dtolnay dtolnay added S-waiting-on-team Status: Awaiting decision from the relevant subteam (see the T-<team> label). and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-libs Relevant to the library team, which will review and decide on the PR/issue. labels Oct 7, 2024
@m-ou-se
Copy link
Member

m-ou-se commented Nov 5, 2024

Discussed in libs-api meeting.

@bors r+

@bors
Copy link
Contributor

bors commented Nov 5, 2024

📌 Commit 6828a1e has been approved by m-ou-se

It is now in the queue for this repository.

@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-team Status: Awaiting decision from the relevant subteam (see the T-<team> label). labels Nov 5, 2024
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this pull request Nov 5, 2024
…r=m-ou-se

Stabilize `UnsafeCell::from_mut`

Closes rust-lang#111645.
FCP: rust-lang#111645 (comment)

Note that because `const_mut_refs` and `const_refs_to_cell` was stabilized, it's okay to const-stabilize this method as well.
@RalfJung
Copy link
Member

RalfJung commented Nov 5, 2024 via email

@RalfJung
Copy link
Member

RalfJung commented Nov 5, 2024 via email

@tgross35 tgross35 removed the I-libs-api-nominated Nominated for discussion during a libs-api team meeting. label Nov 5, 2024
bors added a commit to rust-lang-ci/rust that referenced this pull request Nov 6, 2024
…iaskrgr

Rollup of 5 pull requests

Successful merges:

 - rust-lang#131261 (Stabilize `UnsafeCell::from_mut`)
 - rust-lang#131405 (bootstrap/codegen_ssa: ship llvm-strip and use it for -Cstrip)
 - rust-lang#132077 (Add a new `wide-arithmetic` feature for WebAssembly)
 - rust-lang#132562 (Remove the `wasm32-wasi` target from rustc)
 - rust-lang#132660 (Remove unused errs.rs file)

Failed merges:

 - rust-lang#131721 (Add new unstable feature `const_eq_ignore_ascii_case`)

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit efa5af9 into rust-lang:master Nov 6, 2024
6 checks passed
@rustbot rustbot added this to the 1.84.0 milestone Nov 6, 2024
rust-timer added a commit to rust-lang-ci/rust that referenced this pull request Nov 6, 2024
Rollup merge of rust-lang#131261 - clarfonthey:unsafe-cell-from-mut, r=m-ou-se

Stabilize `UnsafeCell::from_mut`

Closes rust-lang#111645.
FCP: rust-lang#111645 (comment)

Note that because `const_mut_refs` and `const_refs_to_cell` was stabilized, it's okay to const-stabilize this method as well.
@bors
Copy link
Contributor

bors commented Nov 6, 2024

⌛ Testing commit 6828a1e with merge a69df72...

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.

Tracking Issue for UnsafeCell::from_mut
7 participants