Skip to content

NLL: regression test for "dropck: track order of destruction for r-value temporaries" - #54355

Merged
bors merged 1 commit into
rust-lang:masterfrom
pnkfelix:issue-22323-regression-test
Sep 27, 2018
Merged

bors merged 1 commit into
rust-lang:masterfrom
pnkfelix:issue-22323-regression-test

Conversation

@pnkfelix

Copy link
Copy Markdown
Contributor

Once this lands, we can remove the E-needstest from #22323.

(We shouldn't close the bug itself, however, because we are leaving the NLL-fixed-by-NLL bugs open until NLL is turned on by default.)

@rust-highfive

Copy link
Copy Markdown
Contributor

r? @aturon

(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 Sep 19, 2018
@memoryruins memoryruins added the A-NLL Area: Non-lexical lifetimes (NLL) label Sep 19, 2018
@TimNN

TimNN commented Sep 25, 2018

Copy link
Copy Markdown
Contributor

Ping from triage @aturon, this PR requires your review.

@davidtwco

Copy link
Copy Markdown
Member

r? @davidtwco

@rust-highfive rust-highfive assigned davidtwco and unassigned aturon Sep 27, 2018
@davidtwco

Copy link
Copy Markdown
Member

@bors r+

@bors

bors commented Sep 27, 2018

Copy link
Copy Markdown
Collaborator

📌 Commit 3feffd4 has been approved by davidtwco

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

bors commented Sep 27, 2018

Copy link
Copy Markdown
Collaborator

⌛ Testing commit 3feffd4 with merge c9865b1...

bors added a commit that referenced this pull request Sep 27, 2018
NLL: regression test for "dropck: track order of destruction for r-value temporaries"

Once this lands, we can remove the E-needstest from #22323.

(We shouldn't close the bug itself, however, because we are leaving the NLL-fixed-by-NLL bugs open until NLL is turned on by default.)
@bors

bors commented Sep 27, 2018

Copy link
Copy Markdown
Collaborator

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

@bors
bors merged commit 3feffd4 into rust-lang:master Sep 27, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-NLL Area: Non-lexical lifetimes (NLL) 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.

7 participants