Fail build on clippy

indy256 · Jan 19, 2025 · 1449e91 · 1449e91
1 parent 3b3465b
commit 1449e91
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/.github/workflows/rust.yml b/.github/workflows/rust.yml
@@ -14,7 +14,7 @@ jobs:
         run: |
           cd rust
           cargo fmt -- --check
-          RUSTFLAGS="-A unused" cargo clippy
+          RUSTFLAGS="-A unused" cargo clippy -- -Dwarnings
           cargo clippy --tests
       - name: compile and test
         run: |

diff --git a/rust/structures/treap.rs b/rust/structures/treap.rs
@@ -21,7 +21,7 @@ impl<K: Copy + PartialOrd, V: Copy + PartialEq + AddAssign + Ord + Default> Trea
             value,
             mx: value,
             add: V::default(),
-            prio: rand::thread_rng().gen_range(0..1000_000_000),
+            prio: rand::thread_rng().gen_range(0..1_000_000_000),
             size: 1,
             left: None,
             right: None,
@@ -59,6 +59,7 @@ impl<K: Copy + PartialOrd, V: Copy + PartialEq + AddAssign + Ord + Default> Trea
         node.as_ref().map_or(value, |t| max(t.mx, value))
     }
 
+    #[allow(clippy::type_complexity)]
     pub fn split(
         root: Option<Box<Treap<K, V>>>,
         min_right: K,
@@ -73,6 +74,7 @@ impl<K: Copy + PartialOrd, V: Copy + PartialEq + AddAssign + Ord + Default> Trea
         Self::inner_split(root, min_right, |a, b| a > b)
     }
 
+    #[allow(clippy::type_complexity)]
     fn inner_split<F: Fn(K, K) -> bool>(
         root: Option<Box<Treap<K, V>>>,
         min_right: K,
@@ -139,6 +141,7 @@ impl<K: Copy + PartialOrd, V: Copy + PartialEq + AddAssign + Ord + Default> Trea
     ) -> Option<Box<Treap<K, V>>> {
         let (l1, r1) = Self::strict_split(root, rr);
         let (l2, mut r2) = Self::split(l1, ll);
+        #[allow(clippy::option_map_unit_fn)]
         r2.as_mut().map(|t| t.apply(delta));
         Self::merge(Self::merge(l2, r2), r1)
     }