Skip to content

Remove unsafe qualifier from RefCell::as_unsafe_cell #32550

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

Merged
merged 1 commit into from
Apr 1, 2016

Conversation

tbu-
Copy link
Contributor

@tbu- tbu- commented Mar 28, 2016

This method is no longer unsafe because the field of UnsafeCell is no
longer public.

This method is no longer unsafe because the field of `UnsafeCell` is no
longer public.
@rust-highfive
Copy link
Contributor

r? @aturon

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

@alexcrichton alexcrichton added the T-libs-api Relevant to the library API team, which will review and decide on the PR/issue. label Mar 28, 2016
@tbu- tbu- force-pushed the pr_ref_cell_as_unsafe_cell branch from bc051b2 to 4ddbd6b Compare March 28, 2016 20:31
@alexcrichton
Copy link
Member

The libs team discussed this during triage and the decision was to merge. If this is ever going to be stabilized it'll be stabilized like this, so seems prudent to make the change!

@bors: r+ 4ddbd6b

Thanks!

@bors
Copy link
Collaborator

bors commented Apr 1, 2016

⌛ Testing commit 4ddbd6b with merge a2f0cc6...

bors added a commit that referenced this pull request Apr 1, 2016
Remove `unsafe` qualifier from `RefCell::as_unsafe_cell`

This method is no longer unsafe because the field of `UnsafeCell` is no
longer public.
@bors bors merged commit 4ddbd6b into rust-lang:master Apr 1, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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.

5 participants