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

rt: remove unsafe from shell runtime. #2333

Merged
merged 3 commits into from
Mar 21, 2020
Merged

rt: remove unsafe from shell runtime. #2333

merged 3 commits into from
Mar 21, 2020

Conversation

carllerche
Copy link
Member

Since the original shell runtime was implemented, utilities have been
added to encapsulate unsafe. The shell runtime is now able to use
those utilities and not include its own unsafe code.

Since the original shell runtime was implemented, utilities have been
added to encapsulate `unsafe`. The shell runtime is now able to use
those utilities and not include its own `unsafe` code.
Copy link
Member

@hawkw hawkw left a comment

Choose a reason for hiding this comment

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

lgtm — looks like rustfmt needs to be run before merging?

tokio/src/runtime/shell.rs Show resolved Hide resolved
@carllerche carllerche merged commit dd27f1a into master Mar 21, 2020
sthagen added a commit to sthagen/tokio-rs-tokio that referenced this pull request Mar 22, 2020
rt: remove `unsafe` from shell runtime. (tokio-rs#2333)
@carllerche carllerche deleted the rm-unsafe-in-shell branch April 15, 2020 20:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants