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

revert #52991 #53109

Merged
merged 2 commits into from
Aug 7, 2018
Merged

Conversation

nikomatsakis
Copy link
Contributor

Reverts #52991 which is flawed. I have an idea how to fix it but might as well revert first since it is so wildly flawed. That's what I get for opening PRs while on PTO =)

r? @pnkfelix

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Aug 6, 2018
@nikomatsakis nikomatsakis force-pushed the nll-escaping-into-return-revert branch from 294ba81 to 3b4ed18 Compare August 6, 2018 13:00
@nikomatsakis
Copy link
Contributor Author

@bors r+ p=1

I'm going to r+ this myself, since it is just reverting my own work and I don't want to wait. =) Giving high priority because this is a potentially serious NLL bug.

@bors
Copy link
Contributor

bors commented Aug 7, 2018

📌 Commit 3b4ed18 has been approved by nikomatsakis

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Aug 7, 2018
@bors
Copy link
Contributor

bors commented Aug 7, 2018

⌛ Testing commit 3b4ed18 with merge ccb550f...

bors added a commit that referenced this pull request Aug 7, 2018
…r=nikomatsakis

revert #52991

Reverts #52991 which is flawed. I have an idea how to fix it but might as well revert first since it is so wildly flawed. That's what I get for opening PRs while on PTO =)

r? @pnkfelix
@bors
Copy link
Contributor

bors commented Aug 7, 2018

☀️ Test successful - status-appveyor, status-travis
Approved by: nikomatsakis
Pushing ccb550f to master...

@bors bors merged commit 3b4ed18 into rust-lang:master Aug 7, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants