Yield to Clippy's idea of what a number should look like.

bodil · Oct 12, 2018 · 3275e30 · 3275e30
1 parent b0dfd5d
commit 3275e30
Show file tree

Hide file tree

Showing 4 changed files with 18 additions and 16 deletions.
diff --git a/src/hash/map.rs b/src/hash/map.rs
@@ -956,9 +956,11 @@ where
                 None => {
                     out.insert(key, right_value);
                 }
-                Some(left_value) => if let Some(final_value) = f(&key, left_value, right_value) {
-                    out.insert(key, final_value);
-                },
+                Some(left_value) => {
+                    if let Some(final_value) = f(&key, left_value, right_value) {
+                        out.insert(key, final_value);
+                    }
+                }
             }
         }
         out.union(self)
@@ -1946,7 +1948,8 @@ pub mod proptest {
             .prop_map(HashMap::from)
             .prop_filter("Map minimum size".to_owned(), move |m| {
                 m.len() >= size.start
-            }).boxed()
+            })
+            .boxed()
     }
 }
 
@@ -1962,11 +1965,11 @@ mod test {
 
     #[test]
     fn safe_mutation() {
-        let v1: HashMap<usize, usize> = HashMap::from_iter((0..131072).map(|i| (i, i)));
+        let v1: HashMap<usize, usize> = HashMap::from_iter((0..131_072).map(|i| (i, i)));
         let mut v2 = v1.clone();
-        v2.insert(131000, 23);
-        assert_eq!(Some(&23), v2.get(&131000));
-        assert_eq!(Some(&131000), v1.get(&131000));
+        v2.insert(131_000, 23);
+        assert_eq!(Some(&23), v2.get(&131_000));
+        assert_eq!(Some(&131_000), v1.get(&131_000));
     }
 
     #[test]

diff --git a/src/lib.rs b/src/lib.rs
@@ -274,7 +274,6 @@
 
 #![deny(unsafe_code)]
 #![cfg_attr(feature = "cargo-clippy", allow(clippy::type_complexity))]
-#![cfg_attr(feature = "cargo-clippy", allow(clippy::unreadable_literal))]
 #![cfg_attr(has_specialisation, feature(specialization))]
 #![cfg_attr(docs_rs_workaround, feature(slice_get_slice))]
 

diff --git a/src/ord/map.rs b/src/ord/map.rs
@@ -2004,11 +2004,11 @@ mod test {
 
     #[test]
     fn safe_mutation() {
-        let v1 = OrdMap::from_iter((0..131072).map(|i| (i, i)));
+        let v1 = OrdMap::from_iter((0..131_072).map(|i| (i, i)));
         let mut v2 = v1.clone();
-        v2.insert(131000, 23);
-        assert_eq!(Some(&23), v2.get(&131000));
-        assert_eq!(Some(&131000), v1.get(&131000));
+        v2.insert(131_000, 23);
+        assert_eq!(Some(&23), v2.get(&131_000));
+        assert_eq!(Some(&131_000), v1.get(&131_000));
     }
 
     #[test]

diff --git a/src/vector/mod.rs b/src/vector/mod.rs
@@ -2349,20 +2349,20 @@ mod test {
 
     #[test]
     fn large_vector_focus() {
-        let input = Vector::from_iter(0..100000);
+        let input = Vector::from_iter(0..100_000);
         let vec = input.clone();
         let mut sum: i64 = 0;
         let mut focus = vec.focus();
         for i in 0..input.len() {
             sum += *focus.index(i);
         }
-        let expected: i64 = (0..100000).sum();
+        let expected: i64 = (0..100_000).sum();
         assert_eq!(expected, sum);
     }
 
     #[test]
     fn large_vector_focus_mut() {
-        let input = Vector::from_iter(0..100000);
+        let input = Vector::from_iter(0..100_000);
         let mut vec = input.clone();
         {
             let mut focus = vec.focus_mut();