Merge pull request #272 from rex4539/clippy

Fix feature = "cargo-clippy" deprecation

hdf5-sys/src/lib.rs

@@ -1,11 +1,11 @@
 //! Rust bindings to the `hdf5` library for reading and writing data to and from storage
 #![allow(non_camel_case_types, non_snake_case, dead_code, deprecated)]
-#![cfg_attr(feature = "cargo-clippy", allow(clippy::unreadable_literal))]
-#![cfg_attr(feature = "cargo-clippy", allow(clippy::missing_safety_doc))]
-#![cfg_attr(feature = "cargo-clippy", allow(clippy::cognitive_complexity))]
-#![cfg_attr(feature = "cargo-clippy", allow(clippy::upper_case_acronyms))]
-#![cfg_attr(feature = "cargo-clippy", allow(clippy::wildcard_imports))]
-#![cfg_attr(feature = "cargo-clippy", allow(clippy::module_name_repetitions))]
+#![allow(clippy::unreadable_literal)]
+#![allow(clippy::missing_safety_doc)]
+#![allow(clippy::cognitive_complexity)]
+#![allow(clippy::upper_case_acronyms)]
+#![allow(clippy::wildcard_imports)]
+#![allow(clippy::module_name_repetitions)]
 #![cfg_attr(docsrs, feature(doc_cfg))]
 
 macro_rules! extern_static {

hdf5-types/src/lib.rs

@@ -1,8 +1,8 @@
 #![recursion_limit = "1024"]
-#![cfg_attr(feature = "cargo-clippy", allow(clippy::missing_safety_doc))]
-#![cfg_attr(feature = "cargo-clippy", allow(clippy::missing_const_for_fn))]
-#![cfg_attr(feature = "cargo-clippy", allow(clippy::redundant_pub_crate))]
-#![cfg_attr(feature = "cargo-clippy", allow(clippy::must_use_candidate))]
+#![allow(clippy::missing_safety_doc)]
+#![allow(clippy::missing_const_for_fn)]
+#![allow(clippy::redundant_pub_crate)]
+#![allow(clippy::must_use_candidate)]
 
 //! Types that can be stored and retrieved from a `HDF5` dataset
 //!

hdf5/src/lib.rs

@@ -15,33 +15,33 @@
 //! indicating the required version of `HDF5`. The `have-direct` and `have-parallel` features
 //! also indicates `HDF5` functionality.
 
-#![cfg_attr(feature = "cargo-clippy", warn(clippy::pedantic))]
-#![cfg_attr(feature = "cargo-clippy", warn(clippy::nursery))]
-#![cfg_attr(feature = "cargo-clippy", warn(clippy::all))]
-#![cfg_attr(feature = "cargo-clippy", allow(clippy::identity_op))]
-#![cfg_attr(feature = "cargo-clippy", allow(clippy::erasing_op))]
-#![cfg_attr(feature = "cargo-clippy", allow(clippy::cast_sign_loss))]
-#![cfg_attr(feature = "cargo-clippy", allow(clippy::module_name_repetitions))]
-#![cfg_attr(feature = "cargo-clippy", allow(clippy::cast_possible_truncation))]
-#![cfg_attr(feature = "cargo-clippy", allow(clippy::cast_possible_wrap))]
-#![cfg_attr(feature = "cargo-clippy", allow(clippy::cast_precision_loss))]
-#![cfg_attr(feature = "cargo-clippy", allow(clippy::similar_names))]
-#![cfg_attr(feature = "cargo-clippy", allow(clippy::missing_const_for_fn))]
-#![cfg_attr(feature = "cargo-clippy", allow(clippy::missing_safety_doc))]
-#![cfg_attr(feature = "cargo-clippy", allow(clippy::missing_errors_doc))]
-#![cfg_attr(feature = "cargo-clippy", allow(clippy::cognitive_complexity))]
-#![cfg_attr(feature = "cargo-clippy", allow(clippy::must_use_candidate))]
-#![cfg_attr(feature = "cargo-clippy", allow(clippy::wildcard_imports))]
-#![cfg_attr(feature = "cargo-clippy", allow(clippy::struct_excessive_bools))]
-#![cfg_attr(feature = "cargo-clippy", allow(clippy::redundant_pub_crate))]
-#![cfg_attr(feature = "cargo-clippy", allow(clippy::unnecessary_unwrap))]
-#![cfg_attr(feature = "cargo-clippy", allow(clippy::unnecessary_wraps))]
-#![cfg_attr(feature = "cargo-clippy", allow(clippy::upper_case_acronyms))]
-#![cfg_attr(feature = "cargo-clippy", allow(clippy::missing_panics_doc))]
-#![cfg_attr(feature = "cargo-clippy", allow(clippy::missing_const_for_fn))]
-#![cfg_attr(feature = "cargo-clippy", allow(clippy::option_if_let_else))]
-#![cfg_attr(feature = "cargo-clippy", allow(clippy::return_self_not_must_use))]
-#![cfg_attr(all(feature = "cargo-clippy", test), allow(clippy::cyclomatic_complexity))]
+#![allow(clippy::pedantic)]
+#![allow(clippy::nursery)]
+#![allow(clippy::all)]
+#![allow(clippy::identity_op)]
+#![allow(clippy::erasing_op)]
+#![allow(clippy::cast_sign_loss)]
+#![allow(clippy::module_name_repetitions)]
+#![allow(clippy::cast_possible_truncation)]
+#![allow(clippy::cast_possible_wrap)]
+#![allow(clippy::cast_precision_loss)]
+#![allow(clippy::similar_names)]
+#![allow(clippy::missing_const_for_fn)]
+#![allow(clippy::missing_safety_doc)]
+#![allow(clippy::missing_errors_doc)]
+#![allow(clippy::cognitive_complexity)]
+#![allow(clippy::must_use_candidate)]
+#![allow(clippy::wildcard_imports)]
+#![allow(clippy::struct_excessive_bools)]
+#![allow(clippy::redundant_pub_crate)]
+#![allow(clippy::unnecessary_unwrap)]
+#![allow(clippy::unnecessary_wraps)]
+#![allow(clippy::upper_case_acronyms)]
+#![allow(clippy::missing_panics_doc)]
+#![allow(clippy::missing_const_for_fn)]
+#![allow(clippy::option_if_let_else)]
+#![allow(clippy::return_self_not_must_use)]
+#![cfg_attr(all(clippy, test), allow(clippy::cyclomatic_complexity))]
 #![cfg_attr(not(test), allow(dead_code))]
 // To build docs locally:
 // RUSTDOCFLAGS="--cfg docsrs" cargo +nightly doc --features blosc,lzf

hdf5/src/macros.rs

@@ -89,7 +89,7 @@ macro_rules! assert_err_re {
 #[doc(hidden)]
 macro_rules! h5lock {
     ($expr:expr) => {{
-        #[cfg_attr(feature = "cargo-clippy", allow(clippy::redundant_closure))]
+        #[allow(clippy::redundant_closure)]
         #[allow(unused_unsafe)]
         unsafe {
             $crate::sync::sync(|| $expr)