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 <briant@unity3d.com>

src/bin/stdlib/collections/hash_map.rs

@@ -39,12 +39,7 @@ fn main() {
                         BuildTrulyAwfulHasher::new(hash_seed),
                     );
 
-                loop {
-                    let op: Op<u16, u16> = 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

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;