Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

E0207 reports "unconstrained lifetime parameter" for type parameters #35642

Closed
SpaceManiac opened this issue Aug 13, 2016 · 0 comments · Fixed by #35660
Closed

E0207 reports "unconstrained lifetime parameter" for type parameters #35642

SpaceManiac opened this issue Aug 13, 2016 · 0 comments · Fixed by #35660

Comments

@SpaceManiac
Copy link

SpaceManiac commented Aug 13, 2016

E0207 always reads unconstrained lifetime parameter in the advanced diagnostics, regardless of whether the unconstrained parameter is a lifetime or type parameter.

The fix seems simple - substitute format!("unconstrained lifetime parameter") with format!("unconstrained {} parameter", kind) in report_unused_parameter - but I don't have the resources presently to get rustc compiling myself to make a PR.

A test for E0207 added in the fix for #35302 enforces this incorrect behavior and would need to also be updated.

Spotted by Sgeo_ on IRC.

sophiajt pushed a commit to sophiajt/rust that referenced this issue Aug 15, 2016
Update E0207 label to report parameter type

Fixes rust-lang#35642.

r? @jonathandturner (Is this okay? I tagged you since it's a simple error fix similar to the other update ones.)
sophiajt pushed a commit to sophiajt/rust that referenced this issue Aug 17, 2016
Update E0207 label to report parameter type

Fixes rust-lang#35642.

r? @jonathandturner (Is this okay? I tagged you since it's a simple error fix similar to the other update ones.)
sophiajt pushed a commit to sophiajt/rust that referenced this issue Aug 17, 2016
Update E0207 label to report parameter type

Fixes rust-lang#35642.

r? @jonathandturner (Is this okay? I tagged you since it's a simple error fix similar to the other update ones.)
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant