Skip to content

/rust

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

use the infcx tables to check if a closure is Copy #28626

Merged
merged 1 commit into from Oct 2, 2015

Conversation

Reviewers
No reviews
Assignees

@nikomatsakis nikomatsakis

Labels
I-nominated T-compiler beta-accepted
Projects
None yet
Milestone
No milestone
6 participants
@arielb1 @rust-highfive @nikomatsakis @bors @pnkfelix @alexcrichton
@arielb1
Copy link
Contributor

arielb1 commented Sep 24, 2015

Fixes #28550

@rust-highfive rust-highfive assigned nikomatsakis Sep 24, 2015

@rust-highfive

This comment has been minimized.

Sign in to view
Copy link
Collaborator

rust-highfive commented Sep 24, 2015

r? @nikomatsakis

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

This comment has been minimized.

Sign in to view
Copy link
Contributor

nikomatsakis commented Oct 2, 2015

@bors r+
@bors

This comment has been minimized.

Sign in to view
Copy link
Contributor

bors commented Oct 2, 2015

📌 Commit 6b1149d has been approved by nikomatsakis

bors added a commit that referenced this pull request Oct 2, 2015

@bors
Auto merge of #28626 - arielb1:closure-needs-infer, r=nikomatsakis 
Fixes #28550
Loading status checks…
669fc7f
@bors

This comment has been minimized.

Sign in to view
Copy link
Contributor

bors commented Oct 2, 2015

⌛️ Testing commit 6b1149d with merge 669fc7f...

@arielb1 arielb1 added I-nominated beta-nominated T-compiler labels Oct 2, 2015

@arielb1

This comment has been minimized.

Sign in to view
Copy link
Contributor Author

arielb1 commented Oct 2, 2015

cc @rust-lang/compiler
@bors

@bors bors merged commit 6b1149d into rust-lang:master Oct 2, 2015
2 checks passed

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
homu Test successful
Details
@pnkfelix

This comment has been minimized.

Sign in to view
Copy link
Member

pnkfelix commented Oct 16, 2015

@brson I think we should indeed backport this to beta.

(I think we missed it during triage last night ... I'll try to review the persistent urls and see if its accidentally filtering these issues out.)

@nikomatsakis nikomatsakis added the beta-accepted label Oct 16, 2015

@nikomatsakis

This comment has been minimized.

Sign in to view
Copy link
Contributor

nikomatsakis commented Oct 16, 2015

hmm, yes, I agree.

@brson brson referenced this pull request Oct 16, 2015

Merged

Beta next #29112

@alexcrichton alexcrichton removed the beta-nominated label Oct 16, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.