-
Notifications
You must be signed in to change notification settings - Fork 353
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 all uses of to_ptr
#1114
Conversation
☔ The latest upstream changes (presumably #1118) made this pull request unmergeable. Please resolve the merge conflicts. |
@oli-obk what is the plan with this one? |
yea but there's no way to do this by landing something here first. We'll still get the breakage from the rustc PR so we may as well wait for the merger. I'll update this PR though to make all the changes so we can land it immediately |
ceaab16
to
4aef81e
Compare
@bors r+ |
📌 Commit 4aef81e has been approved by |
Remove all uses of `to_ptr` Also only works on top of rust-lang/rust#67192
☀️ Test successful - checks-travis, status-appveyor |
Also only works on top of rust-lang/rust#67192