Fix latest clippy lints

rossmacarthur · Dec 8, 2024 · 0fd1147 · 0fd1147
1 parent 09cb5c6
commit 0fd1147
Show file tree

Hide file tree

Showing 5 changed files with 15 additions and 14 deletions.
diff --git a/crates/macro/Cargo.toml b/crates/macro/Cargo.toml
@@ -10,7 +10,7 @@ repository.workspace = true
 license.workspace = true
 
 [lib]
-proc_macro = true
+proc-macro = true
 
 [dependencies]
 proc-macro2 = "1.0"

diff --git a/crates/stride/src/iter.rs b/crates/stride/src/iter.rs
@@ -45,19 +45,19 @@ impl<'a, T, const S: usize> Iterator for Iter<'a, T, S> {
     }
 }
 
-impl<'a, T, const S: usize> DoubleEndedIterator for Iter<'a, T, S> {
+impl<T, const S: usize> DoubleEndedIterator for Iter<'_, T, S> {
     fn next_back(&mut self) -> Option<Self::Item> {
         self.iter.next_back()
     }
 }
 
-impl<'a, T, const S: usize> ExactSizeIterator for Iter<'a, T, S> {
+impl<T, const S: usize> ExactSizeIterator for Iter<'_, T, S> {
     fn len(&self) -> usize {
         self.iter.len()
     }
 }
 
-impl<'a, T, const S: usize> FusedIterator for Iter<'a, T, S> {}
+impl<T, const S: usize> FusedIterator for Iter<'_, T, S> {}
 
 impl<'a, T, const S: usize> IntoIterator for &'a Stride<T, S> {
     type Item = &'a T;
@@ -110,19 +110,19 @@ impl<'a, T, const S: usize> Iterator for IterMut<'a, T, S> {
     }
 }
 
-impl<'a, T, const S: usize> DoubleEndedIterator for IterMut<'a, T, S> {
+impl<T, const S: usize> DoubleEndedIterator for IterMut<'_, T, S> {
     fn next_back(&mut self) -> Option<Self::Item> {
         self.iter.next_back()
     }
 }
 
-impl<'a, T, const S: usize> ExactSizeIterator for IterMut<'a, T, S> {
+impl<T, const S: usize> ExactSizeIterator for IterMut<'_, T, S> {
     fn len(&self) -> usize {
         self.iter.len()
     }
 }
 
-impl<'a, T, const S: usize> FusedIterator for IterMut<'a, T, S> {}
+impl<T, const S: usize> FusedIterator for IterMut<'_, T, S> {}
 
 impl<'a, T, const S: usize> IntoIterator for &'a mut Stride<T, S> {
     type Item = &'a mut T;

diff --git a/crates/stride/src/lib.rs b/crates/stride/src/lib.rs
@@ -122,7 +122,7 @@ impl<T, const S: usize> Stride<T, S> {
     /// assert_eq!(Stride::<_, 3>::new(data).len(), 2);
     /// ```
     pub const fn len(&self) -> usize {
-        (self.data.len() + S - 1) / S
+        self.data.len().div_ceil(S)
     }
 
     /// Returns `true` if the strided slice has a length of 0.

diff --git a/src/lib.rs b/src/lib.rs
@@ -538,6 +538,7 @@ impl<T, const M: usize, const N: usize> Matrix<T, M, N> {
     /// let column_vector = matrix![1; 2; 3];
     /// assert_eq!(column_vector.l1_norm(), 6);
     /// ```
+    #[doc(alias("manhattan", "taxicab"))]
     pub fn l1_norm(&self) -> T
     where
         T: Copy + Ord + Abs + Zero + Sum<T>,

diff --git a/tests/iter.rs b/tests/iter.rs
@@ -51,10 +51,10 @@ fn matrix_into_iter_fuse() {
     let m = matrix![1, 3, 3, 7];
     let mut iter = m.into_iter();
     for _ in 0..4 {
-        assert!(matches!(iter.next(), Some(_)));
+        assert!(iter.next().is_some());
     }
     for _ in 0..10 {
-        assert!(matches!(iter.next(), None));
+        assert!(iter.next().is_none());
     }
 }
 
@@ -103,10 +103,10 @@ fn matrix_iter_rows_fuse() {
     let m = matrix![1; 3; 3; 7];
     let mut iter = m.iter_rows();
     for _ in 0..4 {
-        assert!(matches!(iter.next(), Some(_)));
+        assert!(iter.next().is_some());
     }
     for _ in 0..10 {
-        assert!(matches!(iter.next(), None));
+        assert!(iter.next().is_none());
     }
 }
 
@@ -115,10 +115,10 @@ fn matrix_iter_columns_fuse() {
     let m = matrix![1, 3, 3, 7];
     let mut iter = m.iter_columns();
     for _ in 0..4 {
-        assert!(matches!(iter.next(), Some(_)));
+        assert!(iter.next().is_some());
     }
     for _ in 0..10 {
-        assert!(matches!(iter.next(), None));
+        assert!(iter.next().is_none());
     }
 }