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

Regression on beta for trait bounds on supertrait's associated items #26813

Closed
Ryman opened this Issue Jul 5, 2015 · 11 comments

Comments

Projects
None yet
7 participants
@Ryman
Copy link
Contributor

Ryman commented Jul 5, 2015

The following playpen works on 1.0, Stable(1.1) and Nightly(1.3 bf3c979 2015-06-30) but fails to compile on current Beta (1.2).

http://is.gd/cQJ1Xk

It would be good if we could backport whatever fixed the behavior for 1.3.

cc @nagisa

@huonw

This comment has been minimized.

Copy link
Member

huonw commented Jul 8, 2015

triage: I-nominated

Seems to be a regression.

@nikomatsakis

This comment has been minimized.

Copy link
Contributor

nikomatsakis commented Jul 9, 2015

It's unclear what's changed here. @nrc do you think this could be the DST coercion stuff? It wasn't obvious to me that it was, but it's a possible candidate, in which case @eddyb was perhaps right ;P.

@nikomatsakis

This comment has been minimized.

Copy link
Contributor

nikomatsakis commented Jul 9, 2015

triage: P-high (regression)

@rust-highfive rust-highfive added P-high and removed I-nominated labels Jul 9, 2015

@nikomatsakis

This comment has been minimized.

Copy link
Contributor

nikomatsakis commented Jul 9, 2015

Not sure who to assign this one too! I'll take it for now.

@nikomatsakis nikomatsakis self-assigned this Jul 9, 2015

@nrc

This comment has been minimized.

Copy link
Member

nrc commented Jul 9, 2015

Seems unlikely to be DST-related to me

@brson

This comment has been minimized.

Copy link
Contributor

brson commented Sep 11, 2015

@nikomatsakis Any new insight into this one? I believe this is about to hit stable in 1.3.

@eddyb

This comment has been minimized.

Copy link
Member

eddyb commented Sep 11, 2015

@brson It works on beta and nightly on the playpen, but not stable, so presumably it will work again in 1.3.

@brson brson added the relnotes label Sep 11, 2015

@brson

This comment has been minimized.

Copy link
Contributor

brson commented Sep 11, 2015

@eddyb Oh, so this is fixed? Can we close?

@eddyb

This comment has been minimized.

Copy link
Member

eddyb commented Sep 12, 2015

cc @nikomatsakis Can you confirm? I'm not entirely sure what the story was here.

@nikomatsakis

This comment has been minimized.

Copy link
Contributor

nikomatsakis commented Sep 14, 2015

I'm not 100% sure what happened here. It sure appears that there was a transient bug which was fixed, but I'm not sure what that bug was. The code appears legit at first glance.

@nikomatsakis

This comment has been minimized.

Copy link
Contributor

nikomatsakis commented Sep 14, 2015

So basically I think yes we can close.

@eddyb eddyb closed this Sep 14, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.