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.

Sign up for GitHub

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

Jump to bottom

Rc<T> does not implement RefUnwindSafe #45924

Closed
mtak- opened this issue Nov 11, 2017 · 0 comments · Fixed by #87467
Closed

Rc<T> does not implement RefUnwindSafe #45924

mtak- opened this issue Nov 11, 2017 · 0 comments · Fixed by #87467
Assignees
@inquisitivecrystal
Labels
C-enhancement Category: An issue proposing an enhancement or a PR with one. T-libs-api Relevant to the library API team, which will review and decide on the PR/issue.

Comments

@mtak-
Copy link
Contributor

mtak- commented Nov 11, 2017

Arc<T> implements both UnwindSafe and RefUnwindSafe (though maybe it only gets RefUnwindSafe because all of it's members happen to implement RefUnwindSafe).

Whereas Rc<T> only implements UnwindSafe. This seems like an oversight.

https://play.rust-lang.org/?gist=0d89773218a347569615e1bea7b352f6&version=nightly
@pietroalbini pietroalbini added C-enhancement Category: An issue proposing an enhancement or a PR with one. T-libs-api Relevant to the library API team, which will review and decide on the PR/issue. labels Jan 23, 2018
@jakoschiko jakoschiko mentioned this issue Oct 30, 2019
Closed
@inquisitivecrystal inquisitivecrystal self-assigned this Jul 22, 2021
@inquisitivecrystal inquisitivecrystal mentioned this issue Jul 26, 2021
Merged
@bors bors closed this as completed in 4061c04 Nov 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@inquisitivecrystal inquisitivecrystal

Labels
C-enhancement Category: An issue proposing an enhancement or a PR with one. T-libs-api Relevant to the library API team, which will review and decide on the PR/issue.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Implement RefUnwindSafe for Rc<T> inquisitivecrystal/rust
3 participants
@pietroalbini @mtak- @inquisitivecrystal