Skip to content

[NLL] Mutability errors#52405

Merged
bors merged 4 commits intorust-lang:masterfrom
matthewjasper:mutability-errors
Jul 21, 2018
Merged

[NLL] Mutability errors#52405
bors merged 4 commits intorust-lang:masterfrom
matthewjasper:mutability-errors

Conversation

@matthewjasper
Copy link
Contributor

@matthewjasper matthewjasper commented Jul 15, 2018

cc #51028
cc #51170
cc #46559
Closes #46629

  • Better explain why the place is immutable ("immutable item" is gone)
  • Distinguish &T and *const T
  • Use better spans when a mutable borrow is for a closure capture

r? @pnkfelix

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jul 15, 2018
@bors
Copy link
Collaborator

bors commented Jul 17, 2018

☔ The latest upstream changes (presumably #52409) made this pull request unmergeable. Please resolve the merge conflicts.

@pnkfelix
Copy link
Contributor

pnkfelix commented Jul 17, 2018

This is great.

pnkfelix
pnkfelix reviewed Jul 17, 2018
View reviewed changes
src/test/ui/borrowck/mutability-errors.rs Outdated
Copy link
Contributor

@pnkfelix pnkfelix Jul 17, 2018

Choose a reason for hiding this comment

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

I personally am a little overwhelmed by the number of cases covered in this file.

I originally wanted to suggest that you break it up into separate files. But I can see from reading the source itself that there is a specific pattern that you are maintaining in the source, and so there is value in keeping all these cases co-located...

@pnkfelix
Copy link
Contributor

pnkfelix commented Jul 17, 2018

okay, r=me once you've resolved the merge conflict.

@pnkfelix pnkfelix added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jul 17, 2018
@matthewjasper
Copy link
Contributor Author

matthewjasper commented Jul 17, 2018

@pnkfelix done.

@pnkfelix
Copy link
Contributor

pnkfelix commented Jul 19, 2018

@bors r+

@bors
Copy link
Collaborator

bors commented Jul 19, 2018

📌 Commit fcfcb3e6012e1eb012009f531f92fca50c7bb645 has been approved by pnkfelix

@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-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Jul 19, 2018
@bors
Copy link
Collaborator

bors commented Jul 19, 2018

☔ The latest upstream changes (presumably #52024) made this pull request unmergeable. Please resolve the merge conflicts.

@bors bors added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Jul 19, 2018
@matthewjasper
MIR changes to improve NLL cannot mutate errors
c3dbdfe 
Alway use unique instead of mutable borrows immutable upvars.
Mark variables that are references for a match guard
@matthewjasper
Move mutability error reporting to its own file
db568e4
@matthewjasper
Improve NLL mutability errors
13b5f69 
* Better explain why the place is immutable
* Distinguish &T and *const T
* Use better spans when a mutable borrow is for a closure capture
@matthewjasper
Update tests for new NLL mutability errors
a06b243
@nikomatsakis
Copy link
Contributor

nikomatsakis commented Jul 21, 2018

@bors r=pnkfelix p=1 -- NLL is important, let's get this landed already :)

@bors
Copy link
Collaborator

bors commented Jul 21, 2018

📌 Commit a06b243 has been approved by pnkfelix

@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-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Jul 21, 2018
@bors
Copy link
Collaborator

bors commented Jul 21, 2018

⌛ Testing commit a06b243 with merge 942b384...

bors added a commit that referenced this pull request Jul 21, 2018
@bors
Auto merge of #52405 - matthewjasper:mutability-errors, r=pnkfelix
942b384 
[NLL] Mutability errors

cc #51028
cc #51170
cc #46559
Closes #46629

* Better explain why the place is immutable ("immutable item" is gone)
* Distinguish &T and *const T
* Use better spans when a mutable borrow is for a closure capture

r? @pnkfelix
@bors
Copy link
Collaborator

bors commented Jul 21, 2018

☀️ Test successful - status-appveyor, status-travis
Approved by: pnkfelix
Pushing 942b384 to master...

@bors bors merged commit a06b243 into rust-lang:master Jul 21, 2018
This was referenced Jul 21, 2018
Merged
Merged
@matthewjasper matthewjasper deleted the mutability-errors branch September 3, 2018 21:32
@eddyb eddyb mentioned this pull request Nov 17, 2020
Merged
@Jules-Bertholet Jules-Bertholet mentioned this pull request Sep 16, 2025
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@pnkfelix pnkfelix pnkfelix left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

@pnkfelix pnkfelix

Labels

S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@matthewjasper @bors @pnkfelix @nikomatsakis @rust-highfive

Comments