Skip to content

adjust closure kind based on the guarantor's upvar note - #45864

Merged
bors merged 2 commits into
rust-lang:masterfrom
nikomatsakis:issue-30046-infer-fn-once-in-closures
Nov 12, 2017
Merged

bors merged 2 commits into
rust-lang:masterfrom
nikomatsakis:issue-30046-infer-fn-once-in-closures

Conversation

@nikomatsakis

Copy link
Copy Markdown
Contributor

Fixes #30046.

r? @eddyb

That encoding that the custom Debugs was using is rather inscrutable,
and incomplete.
This was causing upvar inference to fail for all cases where the move
was from a projection, not the root variable.
@eddyb

eddyb commented Nov 8, 2017

Copy link
Copy Markdown
Contributor

@bors r+

@bors

bors commented Nov 8, 2017

Copy link
Copy Markdown
Collaborator

📌 Commit 629efae has been approved by eddyb

@kennytm kennytm added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Nov 10, 2017
@bors

bors commented Nov 12, 2017

Copy link
Copy Markdown
Collaborator

⌛ Testing commit 629efae with merge c1aacdc...

bors added a commit that referenced this pull request Nov 12, 2017
…ures, r=eddyb

adjust closure kind based on the guarantor's upvar note

Fixes #30046.

r? @eddyb
@bors

bors commented Nov 12, 2017

Copy link
Copy Markdown
Collaborator

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

@bors
bors merged commit 629efae into rust-lang:master Nov 12, 2017
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