Skip to content

Duplicated error messages for static initializers #51970

New issue

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

No Sign up for GitHub

By clicking “No 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? No Sign in to your account

Jump to bottom
Closed
alexreg opened this issue Jul 1, 2018 · 0 comments · Fixed by #52016
Closed

Duplicated error messages for static initializers #51970

alexreg opened this issue Jul 1, 2018 · 0 comments · Fixed by #52016
Assignees
@oli-obk
Labels
A-diagnostics Area: Messages for errors, warnings, and lints

Comments

@alexreg
Copy link
Contributor

alexreg commented Jul 1, 2018 •
edited by estebank
Loading

The error message could not evaluate static initializer is oddly reported twice for some static initializers (see tests src/test/compile-fail/issue-14227.rs and src/test/compile-fail/issue-28324.rs, for example).

This was discovered in #51110, and the suspected cause is a change @oli-obk recently made to error reporting.

CC @oli-obk @eddyb
@estebank estebank added the A-diagnostics Area: Messages for errors, warnings, and lints label Jul 2, 2018
@oli-obk oli-obk self-assigned this Jul 2, 2018
@oli-obk oli-obk mentioned this issue Jul 3, 2018
Merged
kennytm added a commit to kennytm/rust that referenced this issue Jul 6, 2018
@kennytm
Rollup merge of rust-lang#52016 - oli-obk:dedup_static_errors, r=este…
a4846ae 
…bank

Deduplicate error reports for statics

fixes rust-lang#51970
No Sign up for free to join this conversation on GitHub. Already have an account? No Sign in to comment
Assignees

@oli-obk oli-obk

Labels
A-diagnostics Area: Messages for errors, warnings, and lints
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Deduplicate error reports for statics oli-obk/rust
3 participants
@alexreg @oli-obk @estebank