Skip to content

Conversation

@alexcrichton
Copy link
Member

This is a backport of #39466 to beta

This was intended to require `&mut self`, not `&self`, otherwise it's unsound!

Closes rust-lang#39465
@rust-highfive
Copy link
Contributor

r? @aturon

(rust_highfive has picked a reviewer for you, use r? to override)

@rust-highfive
Copy link
Contributor

warning Warning warning

  • Pull requests are usually filed against the master branch for this repo, but this one is against beta. Please double check that you specified the right target!

@aturon
Copy link
Contributor

aturon commented Feb 3, 2017

LGTM. I forget, for beta do we just merge directly?

@alexcrichton
Copy link
Member Author

@aturon nowadays we can just r+!

@aturon
Copy link
Contributor

aturon commented Feb 3, 2017

@bors: r+

@bors
Copy link
Collaborator

bors commented Feb 3, 2017

📌 Commit 2a935f5 has been approved by aturon

@alexcrichton
Copy link
Member Author

@bors: p=1

(PR against beta branch)

@bors
Copy link
Collaborator

bors commented Feb 5, 2017

⌛ Testing commit 2a935f5 with merge 64add42...

@alexcrichton
Copy link
Member Author

@bors: r-

This is gonna fail on beta, need to send another PR

@bors
Copy link
Collaborator

bors commented Feb 5, 2017

💔 Test failed - status-travis

@alexcrichton
Copy link
Member Author

@bors: retry

@alexcrichton
Copy link
Member Author

@bors: r=aturon

@bors
Copy link
Collaborator

bors commented Feb 6, 2017

📌 Commit 2a935f5 has been approved by aturon

@bors
Copy link
Collaborator

bors commented Feb 6, 2017

⌛ Testing commit 2a935f5 with merge 90423c9...

bors added a commit that referenced this pull request Feb 6, 2017
[beta] std: Fix IntoIter::as_mut_slice's signature

This is a backport of #39466 to beta
@bors
Copy link
Collaborator

bors commented Feb 6, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: aturon
Pushing 90423c9 to beta...

@bors bors merged commit 2a935f5 into rust-lang:beta Feb 6, 2017
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.

4 participants