Merge #457

457: Fix clippy false positive r=jonasbb a=jonasbb rust-lang/rust-clippy#8867 bors merge Co-authored-by: Jonas Bushart <jonas@bushart.org>
jonasbb · May 23, 2022 · b3d4156 · b3d4156
2 parents 337b66b + ade8109
commit b3d4156
Show file tree

Hide file tree

Showing 11 changed files with 90 additions and 6 deletions.
diff --git a/serde_with/src/lib.rs b/serde_with/src/lib.rs
@@ -27,10 +27,15 @@
 #![doc(test(no_crate_inject))]
 #![doc(html_root_url = "https://docs.rs/serde_with/1.13.0")]
 #![cfg_attr(docsrs, feature(doc_cfg))]
-// clippy is broken and shows wrong warnings
-// clippy on stable does not know yet about the lint name
-// https://github.com/rust-lang/rust-clippy/issues/8560
-#![allow(unknown_lints, clippy::only_used_in_recursion)]
+#![allow(
+    // clippy is broken and shows wrong warnings
+    // clippy on stable does not know yet about the lint name
+    unknown_lints,
+    // https://github.com/rust-lang/rust-clippy/issues/8560
+    clippy::only_used_in_recursion,
+    // https://github.com/rust-lang/rust-clippy/issues/8867
+    clippy::derive_partial_eq_without_eq,
+)]
 #![no_std]
 
 //! [![crates.io badge](https://img.shields.io/crates/v/serde_with.svg)](https://crates.io/crates/serde_with/)

diff --git a/serde_with/tests/base64.rs b/serde_with/tests/base64.rs
@@ -1,5 +1,12 @@
-// This allows the tests to be written more uniform and not have to special case the last clone().
-#![allow(clippy::redundant_clone)]
+#![allow(
+    // clippy is broken and shows wrong warnings
+    // clippy on stable does not know yet about the lint name
+    unknown_lints,
+    // https://github.com/rust-lang/rust-clippy/issues/8867
+    clippy::derive_partial_eq_without_eq,
+    // This allows the tests to be written more uniform and not have to special case the last clone().
+    clippy::redundant_clone,
+)]
 
 mod utils;
 

diff --git a/serde_with/tests/chrono.rs b/serde_with/tests/chrono.rs
@@ -1,3 +1,11 @@
+#![allow(
+    // clippy is broken and shows wrong warnings
+    // clippy on stable does not know yet about the lint name
+    unknown_lints,
+    // https://github.com/rust-lang/rust-clippy/issues/8867
+    clippy::derive_partial_eq_without_eq,
+)]
+
 extern crate alloc;
 
 mod utils;

diff --git a/serde_with/tests/derives/lib.rs b/serde_with/tests/derives/lib.rs
@@ -1,3 +1,11 @@
+#![allow(
+    // clippy is broken and shows wrong warnings
+    // clippy on stable does not know yet about the lint name
+    unknown_lints,
+    // https://github.com/rust-lang/rust-clippy/issues/8867
+    clippy::derive_partial_eq_without_eq,
+)]
+
 mod deserialize_fromstr;
 mod serialize_display;
 #[path = "../utils.rs"]

diff --git a/serde_with/tests/hex.rs b/serde_with/tests/hex.rs
@@ -1,3 +1,11 @@
+#![allow(
+    // clippy is broken and shows wrong warnings
+    // clippy on stable does not know yet about the lint name
+    unknown_lints,
+    // https://github.com/rust-lang/rust-clippy/issues/8867
+    clippy::derive_partial_eq_without_eq,
+)]
+
 mod utils;
 
 use crate::utils::{check_deserialization, check_error_deserialization, is_equal};

diff --git a/serde_with/tests/indexmap.rs b/serde_with/tests/indexmap.rs
@@ -1,3 +1,11 @@
+#![allow(
+  // clippy is broken and shows wrong warnings
+  // clippy on stable does not know yet about the lint name
+  unknown_lints,
+  // https://github.com/rust-lang/rust-clippy/issues/8867
+  clippy::derive_partial_eq_without_eq,
+)]
+
 mod utils;
 
 use crate::utils::{check_deserialization, check_error_deserialization, is_equal};

diff --git a/serde_with/tests/json.rs b/serde_with/tests/json.rs
@@ -1,3 +1,11 @@
+#![allow(
+    // clippy is broken and shows wrong warnings
+    // clippy on stable does not know yet about the lint name
+    unknown_lints,
+    // https://github.com/rust-lang/rust-clippy/issues/8867
+    clippy::derive_partial_eq_without_eq,
+)]
+
 mod utils;
 
 use crate::utils::is_equal;

diff --git a/serde_with/tests/rust.rs b/serde_with/tests/rust.rs
@@ -1,3 +1,11 @@
+#![allow(
+    // clippy is broken and shows wrong warnings
+    // clippy on stable does not know yet about the lint name
+    unknown_lints,
+    // https://github.com/rust-lang/rust-clippy/issues/8867
+    clippy::derive_partial_eq_without_eq,
+)]
+
 extern crate alloc;
 
 mod utils;

diff --git a/serde_with/tests/serde_as/lib.rs b/serde_with/tests/serde_as/lib.rs
@@ -1,3 +1,11 @@
+#![allow(
+    // clippy is broken and shows wrong warnings
+    // clippy on stable does not know yet about the lint name
+    unknown_lints,
+    // https://github.com/rust-lang/rust-clippy/issues/8867
+    clippy::derive_partial_eq_without_eq,
+)]
+
 extern crate alloc;
 
 mod collections;

diff --git a/serde_with/tests/time_0_3.rs b/serde_with/tests/time_0_3.rs
@@ -1,3 +1,11 @@
+#![allow(
+    // clippy is broken and shows wrong warnings
+    // clippy on stable does not know yet about the lint name
+    unknown_lints,
+    // https://github.com/rust-lang/rust-clippy/issues/8867
+    clippy::derive_partial_eq_without_eq,
+)]
+
 mod utils;
 
 use crate::utils::{check_deserialization, check_error_deserialization, is_equal};

diff --git a/serde_with/tests/with_prefix.rs b/serde_with/tests/with_prefix.rs
@@ -1,3 +1,11 @@
+#![allow(
+    // clippy is broken and shows wrong warnings
+    // clippy on stable does not know yet about the lint name
+    unknown_lints,
+    // https://github.com/rust-lang/rust-clippy/issues/8867
+    clippy::derive_partial_eq_without_eq,
+)]
+
 extern crate alloc;
 
 mod utils;