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

Using trait bounds in default methods fails #7460

Closed
msullivan opened this issue Jun 28, 2013 · 2 comments
Closed

Using trait bounds in default methods fails #7460

msullivan opened this issue Jun 28, 2013 · 2 comments
Labels
A-traits Area: Trait system I-ICE Issue: The compiler panicked, giving an Internal Compilation Error (ICE) ❄️

Comments

@msullivan
Copy link
Contributor

If a trait has parameters with trait bounds, and default methods use them, the compiler ICEs.

This depends on #7266 being fixed and is part of #2794.
@ghost ghost assigned msullivan Jun 28, 2013
msullivan added a commit to msullivan/rust that referenced this issue Jun 29, 2013
@msullivan
Improve handling of trait bounds on a trait in default methods.
0252693 
This is work on rust-lang#7460.
It does not properly handle certain cross crate situations,
because we don't properly export vtable resolution information.
@huonw
Copy link
Member

huonw commented Jul 9, 2013

@msullivan #7266 is closed, is this fixed?

@msullivan
Copy link
Contributor Author

This is "half" fixed. It does not work in certain cross crate cases because the information about vtable resolutions for impls isn't stored properly in cross crate cases.

@msullivan msullivan removed their assignment Jun 16, 2014
flip1995 pushed a commit to flip1995/rust that referenced this issue Jul 19, 2021
@bors
Auto merge of rust-lang#7460 - camsteffen:run-from-source, r=Manishearth
78ffcd9 
Add instructions to run from source

changelog: none

We often get messages on Zulip asking how to install and run Clippy from source. This adds instructions to the readme. I also added a note explaining that `cargo install --path . --force` is bad, which I just decided after some investigating. I use macOS. It would be nice to get some tests on other platforms to see if this is correct.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-traits Area: Trait system I-ICE Issue: The compiler panicked, giving an Internal Compilation Error (ICE) ❄️
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@msullivan @huonw