From 1e8544d9fd5bee80e7c381a658ec78bce6f8224b Mon Sep 17 00:00:00 2001 From: "Brian L. Troutwine" Date: Sun, 9 Sep 2018 08:36:27 -0700 Subject: [PATCH] Back off aggressive use of clippy Clippy has changed its lint names to scoped form. This plays with CI pipelines using stable release until 'tool_lint' feature lands. Seems like this is targeted for the 2018 edition so, uh, this change will surely be reverted in the near term. Signed-off-by: Brian L. Troutwine --- src/bin/stdlib/collections/hash_map.rs | 7 +------ src/lib.rs | 11 ----------- 2 files changed, 1 insertion(+), 17 deletions(-) diff --git a/src/bin/stdlib/collections/hash_map.rs b/src/bin/stdlib/collections/hash_map.rs index c848ca6..8d95a91 100644 --- a/src/bin/stdlib/collections/hash_map.rs +++ b/src/bin/stdlib/collections/hash_map.rs @@ -39,12 +39,7 @@ fn main() { BuildTrulyAwfulHasher::new(hash_seed), ); - loop { - let op: Op = if let Ok(op) = Arbitrary::arbitrary(&mut ring) { - op - } else { - break; - }; + while let Ok(op) = Arbitrary::arbitrary(&mut ring) { match op { Op::Clear => { // Clearning a HashMap removes all elements but keeps diff --git a/src/lib.rs b/src/lib.rs index 836ee87..7d3ba64 100644 --- a/src/lib.rs +++ b/src/lib.rs @@ -7,15 +7,4 @@ extern crate arbitrary; #[deny(rust_2018_compatibility)] #[deny(rust_2018_idioms)] #[deny(unused)] -#[cfg_attr(feature = "cargo-clippy", deny(clippy))] -#[cfg_attr(feature = "cargo-clippy", deny(clippy_pedantic))] -#[cfg_attr(feature = "cargo-clippy", deny(clippy_perf))] -#[cfg_attr(feature = "cargo-clippy", deny(clippy_style))] -#[cfg_attr(feature = "cargo-clippy", deny(clippy_complexity))] -#[cfg_attr(feature = "cargo-clippy", deny(clippy_correctness))] -#[cfg_attr(feature = "cargo-clippy", deny(clippy_cargo))] -// We allow 'stuttering' as the structure of the project will mimic that of -// stdlib. For instance, QC tests for HashMap will appear in a module called -// `hash_map`. -#[cfg_attr(feature = "cargo-clippy", allow(stutter))] pub mod stdlib;