Skip to content

Blacklist fn item types from being used with variadic functions. #32816

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 1 commit into from
Apr 11, 2016

Conversation

eddyb
Copy link
Member

@eddyb eddyb commented Apr 8, 2016

Fixes #32201 by adding fn types to the variadic blacklist which currently includes bool, i8, u8, i16, u16 and f32.

@rust-highfive
Copy link
Contributor

r? @Aatch

(rust_highfive has picked a reviewer for you, use r? to override)

@Aatch
Copy link
Contributor

Aatch commented Apr 8, 2016

@bors r+

@bors
Copy link
Collaborator

bors commented Apr 8, 2016

📌 Commit 942d4c7 has been approved by Aatch

Manishearth added a commit to Manishearth/rust that referenced this pull request Apr 8, 2016
 Fixes rust-lang#32201 by adding fn types to the variadic blacklist which currently includes `bool`, `i8`, `u8`, `i16`, `u16` and `f32`.
Manishearth added a commit to Manishearth/rust that referenced this pull request Apr 10, 2016
Blacklist fn item types from being used with variadic functions.

Fixes rust-lang#32201 by adding fn types to the variadic blacklist which currently includes `bool`, `i8`, `u8`, `i16`, `u16` and `f32`.
@bors
Copy link
Collaborator

bors commented Apr 10, 2016

⌛ Testing commit 942d4c7 with merge 924da29...

bors added a commit that referenced this pull request Apr 10, 2016
Blacklist fn item types from being used with variadic functions.

Fixes #32201 by adding fn types to the variadic blacklist which currently includes `bool`, `i8`, `u8`, `i16`, `u16` and `f32`.
@bors bors merged commit 942d4c7 into rust-lang:master Apr 11, 2016
@eddyb eddyb deleted the variadic-fn-item branch April 11, 2016 04:05
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants