Skip to content

We permit type parameters and so forth on builtin bounds #20302

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
Closed
nikomatsakis opened this issue Dec 29, 2014 · 5 comments · Fixed by #22680
Closed

We permit type parameters and so forth on builtin bounds #20302

nikomatsakis opened this issue Dec 29, 2014 · 5 comments · Fixed by #22680
Labels
E-easy Call for participation: Easy difficulty. Experience needed to fix: Not much. Good first issue. E-mentor Call for participation: This issue has a mentor. Use #t-compiler/help on Zulip for discussion. P-medium Medium priority
Milestone
1.0

Comments

@nikomatsakis
Copy link
Contributor

Example:

fn foo<T:Copy<U>, U>(x: T) {
}

fn main() {
}

I left a FIXME at the relevant part of the code.
@nikomatsakis
Copy link
Contributor Author

Nominating for something.

@brson brson removed the I-nominated label Jan 15, 2015
@brson brson added this to the 1.0 milestone Jan 15, 2015
@brson brson added P-medium Medium priority E-easy Call for participation: Easy difficulty. Experience needed to fix: Not much. Good first issue. labels Jan 15, 2015
@flaper87
Copy link
Contributor

I can mentor...

@flaper87 flaper87 added the E-mentor Call for participation: This issue has a mentor. Use #t-compiler/help on Zulip for discussion. label Jan 23, 2015
@richo
Copy link
Contributor

richo commented Jan 23, 2015

I'm willing to have a look if the mentor offer stands @flaper87

@ahmedcharles
Copy link
Contributor

I already have a proposed fix, I just need to clean it up and add a test.

@flaper87
Copy link
Contributor

@richo thanks for volunteering. The offer is still up but @ahmedcharles has been working on it. I'm happy to help with other bugs too.

@ahmedcharles ahmedcharles mentioned this issue Jan 23, 2015
Closed
This was referenced Feb 22, 2015
Merged
Closed
Manishearth added a commit to Manishearth/rust that referenced this issue Feb 24, 2015
@Manishearth
Rollup merge of rust-lang#22680 - FlaPer87:type_builtin, r=nikomatsakis
4737790 
 Fixes rust-lang#20302
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
E-easy Call for participation: Easy difficulty. Experience needed to fix: Not much. Good first issue. E-mentor Call for participation: This issue has a mentor. Use #t-compiler/help on Zulip for discussion. P-medium Medium priority
Projects
None yet
Milestone
1.0
5 participants
@flaper87 @brson @nikomatsakis @richo @ahmedcharles