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

ICE on ambiguous associated const reference #24922

Closed
quantheory opened this issue Apr 29, 2015 · 0 comments · Fixed by #25065
Closed

ICE on ambiguous associated const reference #24922

quantheory opened this issue Apr 29, 2015 · 0 comments · Fixed by #25065
Labels
I-ICE Issue: The compiler panicked, giving an Internal Compilation Error (ICE) ❄️

Comments

@quantheory
Copy link
Contributor

I plan to fix this one very soon because it is easy, but I thought that it deserved a bug report in any case. Note the comment by @nikomatsakis here with a test case.
@steveklabnik steveklabnik added the I-ICE Issue: The compiler panicked, giving an Internal Compilation Error (ICE) ❄️ label Apr 30, 2015
@quantheory quantheory mentioned this issue May 3, 2015
Merged
bors added a commit that referenced this issue May 14, 2015
@bors
Auto merge of #25065 - quantheory:fix_associated_const_ambiguity_mess…
1b5b639 
…age, r=nikomatsakis

This fixes #24922 and #25017, and reduces the number of error messages that talk about "methods" when associated constants rather than methods are involved.

I will admit that I haven't thought very carefully about the error messages. My goal has been to make more of the messages technically correct in all situations, and to avoid ICEs. But in some cases we could probably talk specifically about "methods" rather than "items".
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
I-ICE Issue: The compiler panicked, giving an Internal Compilation Error (ICE) ❄️
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix some simple associated const issues. quantheory/rust
2 participants
@steveklabnik @quantheory