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.

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

Referencing fn pointers with associated types of references ICEs #38381

Closed
sgrif opened this issue Dec 15, 2016 · 4 comments
Closed

Referencing fn pointers with associated types of references ICEs #38381

sgrif opened this issue Dec 15, 2016 · 4 comments
Labels
E-needs-test Call for participation: An issue has been fixed and does not reproduce, but no test has been added.

Comments

@sgrif
Copy link
Contributor

sgrif commented Dec 15, 2016

Example reproduction script:

use std::ops::Deref;

fn main() {
    let x: fn(&i32) -> <&i32 as Deref>::Target = unimplemented!();
}
@mzji
Copy link

mzji commented Dec 15, 2016

Looks like this is fixed by #38375 . @petrochenkov ?

@petrochenkov
Copy link
Contributor

No, this is some other older issue, it happens on stable already.

@mzji
Copy link

mzji commented Dec 15, 2016

Oh. Thanks for the explain @petrochenkov 😄

@Mark-Simulacrum
Copy link
Member

This no longer ICEs. E-needstest.

@Mark-Simulacrum Mark-Simulacrum added the E-needs-test Call for participation: An issue has been fixed and does not reproduce, but no test has been added. label May 18, 2017
Mark-Simulacrum added a commit to Mark-Simulacrum/rust that referenced this issue Jun 22, 2017
Add tests for a few issues.

Fixes rust-lang#41998
Fixes rust-lang#38381
Fixes rust-lang#37515
Fixes rust-lang#37510
Fixes rust-lang#37508
Fixes rust-lang#37366
Fixes rust-lang#37323
Fixes rust-lang#37051
Fixes rust-lang#36839
Fixes rust-lang#35570
Fixes rust-lang#34373
Fixes rust-lang#34222

Certainly not all of the E-needstest issues right now, but I started to get bored.
bors added a commit that referenced this issue Jun 24, 2017
Add tests for a few issues.

Fixes #41998
Fixes #38381
Fixes #37515
Fixes #37510
Fixes #37366
Fixes #37323
Fixes #37051
Fixes #36839
Fixes #35570
Fixes #34373
Fixes #34222

Certainly not all of the E-needstest issues right now, but I started to get bored.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
E-needs-test Call for participation: An issue has been fixed and does not reproduce, but no test has been added.
Projects
None yet
Development

No branches or pull requests

4 participants