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

Add a test for #48070 #48988

Merged
merged 2 commits into from
Mar 14, 2018
Merged

Add a test for #48070 #48988

merged 2 commits into from
Mar 14, 2018

Conversation

sapphire-arches
Copy link
Contributor

Resolves #48070.
The bug itself was fixed by #48770, but that PR didn't add a test for it.

r? @nikomatsakis

Resolves rust-lang#48070.
The bug itself was fixed by rust-lang#48770, but that PR didn't add a test for it.
@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Mar 13, 2018
@@ -0,0 +1,34 @@
// Copyright 2018 The Rust Project Developers. See the COPYRIGHT
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can you rename this file to run-pass/nll/issue-48070.rs?

Or, better yet, ui/nll/issue-48070.rs? (In that case, you would add a // run-pass comment in the header)

@sapphire-arches
Copy link
Contributor Author

I have moved the test to ui/nll/ and added the run-pass tag.

@nikomatsakis
Copy link
Contributor

@bors r+ rollup

@bors
Copy link
Contributor

bors commented Mar 13, 2018

📌 Commit 52bb99d 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 Mar 13, 2018
kennytm added a commit to kennytm/rust that referenced this pull request Mar 14, 2018
…atsakis

Add a test for rust-lang#48070

Resolves rust-lang#48070.
The bug itself was fixed by rust-lang#48770, but that PR didn't add a test for it.

r? @nikomatsakis
bors added a commit that referenced this pull request Mar 14, 2018
Rollup of 12 pull requests

- Successful merges: #48765, #48831, #48840, #48964, #48970, #48971, #48981, #48988, #48991, #48966, #48993, #48874
- Failed merges:
@bors bors merged commit 52bb99d into rust-lang:master Mar 14, 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.

two-phase borrows are causing #[feature(nll)] to reject a mutable borrow incorrectly
4 participants