Skip to content

NLL migration in the 2018 edition needs two-phase borrows too! - #52975

Merged
bors merged 1 commit into
rust-lang:masterfrom
pnkfelix:issue-52967-edition-2018-implies-2-phase-borrows
Aug 2, 2018
Merged

bors merged 1 commit into
rust-lang:masterfrom
pnkfelix:issue-52967-edition-2018-implies-2-phase-borrows

Conversation

@pnkfelix

@pnkfelix pnkfelix commented Aug 2, 2018

Copy link
Copy Markdown
Contributor

NLL migration in the 2018 edition needs two-phase borrows too!

Fix #52967.

@rust-highfive

Copy link
Copy Markdown
Contributor

r? @eddyb

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

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Aug 2, 2018
@Mark-Simulacrum

Copy link
Copy Markdown
Member

@bors r+ p=5

@bors

bors commented Aug 2, 2018

Copy link
Copy Markdown
Collaborator

📌 Commit 09a7e64 has been approved by Mark-Simulacrum

@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 2, 2018
@bors

bors commented Aug 2, 2018

Copy link
Copy Markdown
Collaborator

⌛ Testing commit 09a7e64 with merge 5e92bf2...

bors added a commit that referenced this pull request Aug 2, 2018
…ase-borrows, r=Mark-Simulacrum

NLL migration in the 2018 edition needs two-phase borrows too!

NLL migration in the 2018 edition needs two-phase borrows too!

Fix #52967.
@bors

bors commented Aug 2, 2018

Copy link
Copy Markdown
Collaborator

☀️ Test successful - status-appveyor, status-travis
Approved by: Mark-Simulacrum
Pushing 5e92bf2 to master...

@bors
bors merged commit 09a7e64 into rust-lang:master Aug 2, 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.

5 participants