Skip to content

Commit 91de22e

Browse files
committed
Fix clippy warnings in cust_raw
- Remove derive_ord from bindgen (incompatible with function pointers) - Add no_partialeq/no_hash for types with function pointers - Allow missing_safety_doc for bindgen-generated modules
1 parent 4dacb99 commit 91de22e

2 files changed

Lines changed: 7 additions & 4 deletions

File tree

crates/cust_raw/build/main.rs

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -117,7 +117,6 @@ fn create_cuda_driver_bindings(
117117
let header = manifest_dir.join("build/driver_wrapper.h");
118118
println!("cargo::rerun-if-changed={}", header.display());
119119
let bindings = bindgen::Builder::default()
120-
.raw_line("#![allow(clippy::missing_safety_doc)]")
121120
.header(header.to_str().expect("header should be valid UTF-8"))
122121
.parse_callbacks(Box::new(
123122
callbacks::BindgenCallbacks::with_function_renames(callbacks::FunctionRenames::new(
@@ -173,7 +172,6 @@ fn create_cuda_runtime_bindings(
173172
let header = manifest_dir.join("build/runtime_wrapper.h");
174173
println!("cargo::rerun-if-changed={}", header.display());
175174
let bindings = bindgen::Builder::default()
176-
.raw_line("#![allow(clippy::missing_safety_doc)]")
177175
.header(header.to_str().expect("header should be valid UTF-8"))
178176
.parse_callbacks(Box::new(
179177
callbacks::BindgenCallbacks::with_function_renames(callbacks::FunctionRenames::new(
@@ -273,7 +271,6 @@ fn create_nptx_compiler_bindings(
273271
let header = manifest_dir.join("build/nvptx_compiler_wrapper.h");
274272
println!("cargo::rerun-if-changed={}", header.display());
275273
let bindings = bindgen::Builder::default()
276-
.raw_line("#![allow(clippy::missing_safety_doc)]")
277274
.header(header.to_str().expect("header should be valid UTF-8"))
278275
.parse_callbacks(Box::new(callbacks::BindgenCallbacks::simple()))
279276
.clang_args(
@@ -308,7 +305,6 @@ fn create_nvvm_bindings(sdk: &cuda_sdk::CudaSdk, outdir: &path::Path, manifest_d
308305
let header = manifest_dir.join("build/nvvm_wrapper.h");
309306
println!("cargo::rerun-if-changed={}", header.display());
310307
let bindings = bindgen::Builder::default()
311-
.raw_line("#![allow(clippy::missing_safety_doc)]")
312308
.header(header.to_str().expect("header should be valid UTF-8"))
313309
.parse_callbacks(Box::new(callbacks::BindgenCallbacks::simple()))
314310
.clang_args(

crates/cust_raw/src/lib.rs

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,16 +1,23 @@
11
#[cfg(feature = "driver")]
2+
#[allow(clippy::missing_safety_doc)]
23
pub mod driver_sys;
34
#[cfg(feature = "runtime")]
5+
#[allow(clippy::missing_safety_doc)]
46
pub mod runtime_sys;
57

68
#[cfg(feature = "cublas")]
9+
#[allow(clippy::missing_safety_doc)]
710
pub mod cublas_sys;
811
#[cfg(feature = "cublaslt")]
12+
#[allow(clippy::missing_safety_doc)]
913
pub mod cublaslt_sys;
1014
#[cfg(feature = "cublasxt")]
15+
#[allow(clippy::missing_safety_doc)]
1116
pub mod cublasxt_sys;
1217

1318
#[cfg(feature = "nvptx-compiler")]
19+
#[allow(clippy::missing_safety_doc)]
1420
pub mod nvptx_compiler_sys;
1521
#[cfg(feature = "nvvm")]
22+
#[allow(clippy::missing_safety_doc)]
1623
pub mod nvvm_sys;

0 commit comments

Comments
 (0)