Skip to content

implement CovariantUnsafeCell#159738

Open
WaffleLapkin wants to merge 2 commits into
rust-lang:mainfrom
WaffleLapkin:covariant-unsafe-cell
Open

implement CovariantUnsafeCell#159738
WaffleLapkin wants to merge 2 commits into
rust-lang:mainfrom
WaffleLapkin:covariant-unsafe-cell

Conversation

@WaffleLapkin

Copy link
Copy Markdown
Member

@rustbot

This comment was marked as resolved.

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. T-libs Relevant to the library team, which will review and decide on the PR/issue. labels Jul 22, 2026
@rustbot

rustbot commented Jul 22, 2026

Copy link
Copy Markdown
Collaborator

r? @jhpratt

rustbot has assigned @jhpratt.
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

Why was this reviewer chosen?

The reviewer was selected based on:

  • Owners of files modified in this PR: libs
  • libs expanded to 12 candidates
  • Random selection from 6 candidates

#[rustc_should_not_be_called_on_const_items]
#[unstable(feature = "covariant_unsafe_cell", issue = "159735")]
#[rustc_const_unstable(feature = "covariant_unsafe_cell", issue = "159735")]
pub const fn get(&self) -> NonNull<T> {

@WaffleLapkin WaffleLapkin Jul 22, 2026

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Minor divergence from the ACP: returning NonNull<T> instead of *mut T.

View changes since the review

@rust-log-analyzer

This comment has been minimized.

@WaffleLapkin
WaffleLapkin force-pushed the covariant-unsafe-cell branch 2 times, most recently from b2a7ea2 to c37239a Compare July 22, 2026 19:53
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. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. T-libs Relevant to the library team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants