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

treat for<'a> T: 'a as T: 'static #27096

Merged
merged 2 commits into from
Jul 18, 2015
Merged

treat for<'a> T: 'a as T: 'static #27096

merged 2 commits into from
Jul 18, 2015

Conversation

apasel422
Copy link
Contributor

closes #26217

r? @nikomatsakis

@apasel422
Copy link
Contributor Author

@nikomatsakis I think I implemented this as you suggested to me, but the error message for types that fail the 'static obligation here should be changed, as it is not necessarily obvious how the for<'a> T: 'a predicate becomes 'static. Additionally, in order to make sure the logic is sound, I'm curious how we ultimately might support the following:

fn foo<'a, T>() where for<'b: 'a> T: 'b {}

@nikomatsakis
Copy link
Contributor

@apasel422 well, adding where clauses to for binders is a much larger topic =)

@nikomatsakis
Copy link
Contributor

@bors r+

@bors
Copy link
Contributor

bors commented Jul 17, 2015

📌 Commit d088db9 has been approved by nikomatsakis

@bors
Copy link
Contributor

bors commented Jul 18, 2015

⌛ Testing commit d088db9 with merge a27fed7...

bors added a commit that referenced this pull request Jul 18, 2015
@bors bors merged commit d088db9 into rust-lang:master Jul 18, 2015
@apasel422 apasel422 deleted the issue-26217 branch July 18, 2015 13:21
@apasel422
Copy link
Contributor Author

What issues should I open as follow-ups to this PR?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Higher-kinded lifetime bounds could be smarter
3 participants