search

luqmana / rust-opencl

OpenCL bindings for Rust.
Other
170 stars 42 forks source link

Update to latest rust. #32

Closed ghost closed 10 years ago

eholk commented 10 years ago

Thanks!

At some future refactoring, we should replace the types with CamelCase if possible so we can follow the default compiler settings better.

ghost commented 10 years ago

There are a few types like Array2D_cl that should be renamed. But most of the warnings were from things like this

src/OpenCL/CL.rs:33:1: 33:27 warning: type `cl_long` should have a camel case identifier, #[warn(non_camel_case_types)] on by default
src/OpenCL/CL.rs:33 pub type cl_long    = i64;

Which is just the name from the Kronos spec. So there is an argument for keeping them named as is.