Skip to content

use Memory::read_c_str, avoid a few to_ptr #824

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 1 commit into from
Jul 6, 2019
Merged

Conversation

RalfJung
Copy link
Member

@RalfJung RalfJung commented Jul 5, 2019

This is the Miri side of rust-lang/rust#62257

@RalfJung RalfJung added the S-blocked-on-rust Status: Blocked on landing a Rust PR label Jul 5, 2019
@bors
Copy link
Contributor

bors commented Jul 5, 2019

☔ The latest upstream changes (presumably #825) made this pull request unmergeable. Please resolve the merge conflicts.

@RalfJung RalfJung removed the S-blocked-on-rust Status: Blocked on landing a Rust PR label Jul 6, 2019
@RalfJung
Copy link
Member Author

RalfJung commented Jul 6, 2019

@bors r+

@bors
Copy link
Contributor

bors commented Jul 6, 2019

📌 Commit 47bfc62 has been approved by RalfJung

@bors
Copy link
Contributor

bors commented Jul 6, 2019

⌛ Testing commit 47bfc62 with merge e734f12...

bors added a commit that referenced this pull request Jul 6, 2019
use Memory::read_c_str, avoid a few to_ptr

This is the Miri side of rust-lang/rust#62257
@bors
Copy link
Contributor

bors commented Jul 6, 2019

☀️ Test successful - checks-travis, status-appveyor
Approved by: RalfJung
Pushing e734f12 to master...

@bors bors merged commit 47bfc62 into rust-lang:master Jul 6, 2019
@RalfJung RalfJung deleted the c_str branch July 26, 2019 08:16
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants