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

Remove explicit negative Send and Sync impls #7

Merged
merged 1 commit into from
Jul 23, 2019
Merged

Conversation

lopopolo
Copy link
Member

Rc and Weak are already not Send and Sync because RcBox includes Cell
and RefCell members.

Remove the negative trait impls which allows removing the nightly feature
optin_builtin_traits. Add doctests to ensure that Rc and Weak are !Send
and !Sync.

Rc and Weak are already not Send and Sync because RcBox includes Cell
and RefCell members.

Remove the negative trait impls which allows removing the nightly feature
optin_builtin_traits. Add doctests to ensure that Rc and Weak are !Send
and !Sync.
@lopopolo lopopolo merged commit d9388fa into master Jul 23, 2019
@lopopolo lopopolo deleted the send-sync branch July 23, 2019 03:41
@lopopolo lopopolo added the A-unsafe-code Area: Unsafe code, be careful when reviewing. label Jun 13, 2021
@lopopolo
Copy link
Member Author

This change needs to be reverted because Rc and Weak only include an *mut RcBox<T> which is Copy because it is a pointer.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-unsafe-code Area: Unsafe code, be careful when reviewing.
Development

Successfully merging this pull request may close these issues.

None yet

1 participant