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

make Alloc::check_bounds_ptr private; you should use Memory::check_bounds_ptr instead #57129

Merged
merged 1 commit into from Dec 27, 2018

Conversation

Projects
None yet
4 participants
@RalfJung
Copy link
Member

RalfJung commented Dec 26, 2018

@oli-obk

This comment has been minimized.

Copy link
Contributor

oli-obk commented Dec 26, 2018

@bors r+ rollup

@bors

This comment has been minimized.

Copy link
Contributor

bors commented Dec 26, 2018

📌 Commit fd69e6e has been approved by oli-obk

@bors

This comment has been minimized.

Copy link
Contributor

bors commented Dec 26, 2018

🌲 The tree is currently closed for pull requests below priority 500, this pull request will be tested once the tree is reopened

@bors

This comment has been minimized.

Copy link
Contributor

bors commented Dec 27, 2018

⌛️ Testing commit fd69e6e with merge 39bc8f5...

bors added a commit that referenced this pull request Dec 27, 2018

Auto merge of #57129 - RalfJung:check-bounds, r=oli-obk
make Alloc::check_bounds_ptr private; you should use Memory::check_bounds_ptr instead

r? @oli-obk
@bors

This comment has been minimized.

Copy link
Contributor

bors commented Dec 27, 2018

☀️ Test successful - status-appveyor, status-travis
Approved by: oli-obk
Pushing 39bc8f5 to master...

@bors bors merged commit fd69e6e into rust-lang:master Dec 27, 2018

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
homu Test successful
Details

@RalfJung RalfJung referenced this pull request Dec 27, 2018

Merged

Update cargo, rls, miri #56924

@RalfJung RalfJung deleted the RalfJung:check-bounds branch Jan 9, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment