fix: remove or move allow attributes

benches/movingai.rs

@@ -1,14 +1,13 @@
 // Test with files from https://movingai.com/benchmarks/
 
-#![allow(clippy::cast_precision_loss)]
-
 use codspeed_criterion_compat::{criterion_group, criterion_main, Criterion};
 use movingai::parser::{parse_map_file, parse_scen_file};
 use movingai::{Coords2D, Map2D};
 use noisy_float::prelude::*;
 use pathfinding::directed::astar::astar;
 use std::path::Path;
 
+#[allow(clippy::cast_precision_loss)]
 fn distance(a: &Coords2D, b: &Coords2D) -> R64 {
     r64((a.0 as f64 - b.0 as f64).hypot(a.1 as f64 - b.1 as f64))
 }

examples/sliding-puzzle.rs

@@ -1,5 +1,3 @@
-#![allow(clippy::cast_possible_truncation)]
-
 use itertools::Itertools;
 use lazy_static::lazy_static;
 use pathfinding::prelude::{astar, idastar};
@@ -37,7 +35,7 @@ lazy_static! {
         positions: {
             let mut p = [0u8; LIMIT];
             for (i, e) in p.iter_mut().enumerate() {
-                *e = i as u8;
+                *e = u8::try_from(i).unwrap();
             }
             p
         },
@@ -125,7 +123,8 @@ impl Game {
     }
 
     fn from_array(positions: [u8; LIMIT]) -> Game {
-        let hole_idx = positions.iter().find_position(|&&n| n == 0).unwrap().0 as u8;
+        let hole_idx =
+            u8::try_from(positions.iter().find_position(|&&n| n == 0).unwrap().0).unwrap();
         let mut game = Game {
             positions,
             hole_idx,

src/directed/edmonds_karp.rs

@@ -14,7 +14,6 @@ use std::collections::{BTreeMap, BTreeSet, VecDeque};
 use std::hash::Hash;
 
 /// Type alias for Edmonds-Karp maximum flow result.
-#[allow(clippy::upper_case_acronyms)]
 pub type EKFlows<N, C> = (Vec<Edge<N, C>>, C, Vec<Edge<N, C>>);
 
 /// Type alias for representing an edge in a graph

src/matrix.rs

@@ -86,7 +86,6 @@ impl<C: Clone> Matrix<C> {
     ///
     /// [`MatrixFormatError::WrongIndex`] if the ranges
     /// are outside the original matrix.
-    #[allow(clippy::needless_pass_by_value)]
     pub fn slice(
         &self,
         rows: Range<usize>,

src/utils.rs

@@ -13,11 +13,6 @@ use num_traits::{PrimInt, Unsigned};
 /// assert_eq!(uint_sqrt(100usize), Some(10));
 /// assert_eq!(uint_sqrt(10usize), None);
 /// ```
-#[allow(
-    clippy::cast_possible_truncation,
-    clippy::cast_precision_loss,
-    clippy::cast_sign_loss
-)]
 pub fn uint_sqrt<T>(n: T) -> Option<T>
 where
     T: PrimInt + Unsigned,

tests/codejam-2017-a.rs

@@ -1,8 +1,6 @@
 // Problem A from the Google Code Jam finals 2017.
 // https://code.google.com/codejam/contest/dashboard?c=6314486#s=p0&a=0
 
-#![allow(clippy::cast_sign_loss)]
-
 use pathfinding::prelude::*;
 use std::collections::{BTreeMap, BTreeSet, HashMap, HashSet};
 use std::io::prelude::*;
@@ -101,6 +99,7 @@ fn test<EK: EdmondsKarp<i32>>(n: usize, file: &mut dyn BufRead) -> Result<String
             loop {
                 let (_, n, _) = ek.augment();
                 debug_assert!(n >= 0);
+                #[allow(clippy::cast_sign_loss)]
                 let n = n as usize;
                 if n > max {
                     max = n;

tests/gps.rs

@@ -1,5 +1,4 @@
 #![cfg(test)]
-#![allow(clippy::cast_possible_truncation, clippy::cast_sign_loss)]
 
 use pathfinding::prelude::*;
 use std::collections::HashMap;
@@ -16,6 +15,7 @@ impl Coords {
         self.1.to_radians()
     }
 
+    #[allow(clippy::cast_possible_truncation, clippy::cast_sign_loss)]
     fn distance_in_meters(&self, other: &Coords) -> u64 {
         let x =
             (other.lon_rad() - self.lon_rad()) * ((other.lat_rad() + self.lat_rad()) / 2.0).cos();