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.

Sign up for GitHub

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

Jump to bottom

Remove the implicit binder from the FnSpace in VecPerParamSpace #20526

Closed
nikomatsakis opened this issue Jan 4, 2015 · 1 comment
Closed

Remove the implicit binder from the FnSpace in VecPerParamSpace #20526

nikomatsakis opened this issue Jan 4, 2015 · 1 comment
Labels
A-type-system Area: Type system C-cleanup Category: PRs that clean code up or issues documenting cleanup.

Comments

@nikomatsakis
Copy link
Contributor

Right now there are two region binders in ty_fold: Binder<T>, and the FnSpace in VecPerParamSpace. There should be only one: Binder<T>. In fact, because of the early/late system, late-bound regions from a method or what have you cannot appear in the FnSpace predicates, but we keep this extra binding level because of the way that resolve_lifetime is structured. We should remove it.
@kmcallister kmcallister added C-cleanup Category: PRs that clean code up or issues documenting cleanup. A-type-system Area: Type system labels Jan 14, 2015
@jseyfried jseyfried mentioned this issue Oct 29, 2015
Merged
@steveklabnik
Copy link
Member

Triage: looks like #29463 is stuck in review, but should fix this.

@bors bors closed this as completed in 7ded11a Apr 5, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-type-system Area: Type system C-cleanup Category: PRs that clean code up or issues documenting cleanup.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@steveklabnik @nikomatsakis @kmcallister