Skip to content

E0107 needs to be updated to new format #35246

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

Jump to bottom
Closed
sophiajt opened this issue Aug 3, 2016 · 0 comments
Closed

E0107 needs to be updated to new format #35246

sophiajt opened this issue Aug 3, 2016 · 0 comments
Labels
A-diagnostics Area: Messages for errors, warnings, and lints E-easy Call for participation: Easy difficulty. Experience needed to fix: Not much. Good first issue. E-mentor Call for participation: This issue has a mentor. Use #t-compiler/help on Zulip for discussion.

Comments

@sophiajt
Copy link
Contributor

sophiajt commented Aug 3, 2016

From: src/test/compile-fail/E0107.rs

Error E0107 needs a span_label, updating it from:

error[E0107]: wrong number of lifetime parameters: expected 1, found 0
  --> src/test/compile-fail/E0107.rs:20:10
   |
20 |     foo: Foo, //~ ERROR E0107
   |          ^^^

error[E0107]: wrong number of lifetime parameters: expected 0, found 1
  --> src/test/compile-fail/E0107.rs:21:10
   |
21 |     bar: Bar<'a>, //~ ERROR E0107
   |          ^^^^^^^

To:

error[E0107]: wrong number of lifetime parameters: expected 1, found 0
  --> src/test/compile-fail/E0107.rs:20:10
   |
20 |     foo: Foo, //~ ERROR E0107
   |          ^^^ expected 1 lifetime parameter

error[E0107]: wrong number of lifetime parameters: expected 0, found 1
  --> src/test/compile-fail/E0107.rs:21:10
   |
21 |     bar: Bar<'a>, //~ ERROR E0107
   |          ^^^^^^^ unexpected lifetime parameter

This assumes we can see how many lifetime parameters we expect, and update the label accordingly.
@sophiajt sophiajt added E-easy Call for participation: Easy difficulty. Experience needed to fix: Not much. Good first issue. A-diagnostics Area: Messages for errors, warnings, and lints E-mentor Call for participation: This issue has a mentor. Use #t-compiler/help on Zulip for discussion. labels Aug 3, 2016
@GuillaumeGomez GuillaumeGomez mentioned this issue Aug 3, 2016
Closed
@ojsheikh ojsheikh mentioned this issue Aug 5, 2016
Merged
sophiajt pushed a commit to sophiajt/rust that referenced this issue Aug 5, 2016
Rollup merge of rust-lang#35373 - oijazsh:E0107, r=jonathandturner
7f5d263 
Update E0107 message to new format

Fixes rust-lang#35246 as part of rust-lang#35233.

r? @jonathandturner
eddyb added a commit to eddyb/rust that referenced this issue Aug 6, 2016
@eddyb
Rollup merge of rust-lang#35373 - oijazsh:E0107, r=jonathandturner
b053da3 
Update E0107 message to new format

Fixes rust-lang#35246 as part of rust-lang#35233.

r? @jonathandturner
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees
No one assigned
Labels
A-diagnostics Area: Messages for errors, warnings, and lints E-easy Call for participation: Easy difficulty. Experience needed to fix: Not much. Good first issue. E-mentor Call for participation: This issue has a mentor. Use #t-compiler/help on Zulip for discussion.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@sophiajt