Use last_mut()? instead of indexing

rust-itertools · Jan 16, 2025 · 5a7c522 · 5a7c522
1 parent 3dd45f5
commit 5a7c522
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/src/permutations.rs b/src/permutations.rs
@@ -84,7 +84,9 @@ where
             PermutationState::Buffered { indices, min_n } => {
                 let k = indices.len();
                 if vals.get_next() {
-                    indices.borrow_mut()[k.value() - 1] += 1;
+                    // This could be an unwrap instead of ?, indices.len() should be
+                    // k.value() > 0 to be in this case
+                    *indices.borrow_mut().last_mut()? += 1;
                     debug_assert!(indices
                         .borrow()
                         .iter()