iter mut

spiraldb · Sep 15, 2024 · ac438f8 · robert3005 · Sep 15, 2024 · ac438f8
1 parent ffaf0c7
commit ac438f8
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 8 deletions.
diff --git a/vortex-array/src/array/chunked/compute/filter.rs b/vortex-array/src/array/chunked/compute/filter.rs
@@ -126,9 +126,8 @@ fn filter_slices<'a>(
                 ChunkFilter::Slices(slices) => slices.push(end_slice),
             }
 
-            #[allow(clippy::needless_range_loop)]
-            for chunk in (start_chunk + 1)..end_chunk {
-                chunk_filters[chunk] = ChunkFilter::All;
+            for chunk in &mut chunk_filters[start_chunk + 1..end_chunk] {
+                *chunk = ChunkFilter::All;
             }
         }
     }

diff --git a/vortex-array/src/array/chunked/mod.rs b/vortex-array/src/array/chunked/mod.rs
@@ -168,11 +168,6 @@ impl ChunkedArray {
 /// A tuple (chunk_id, chunk_offset) that encodes the chunk and the offset within chunk
 /// where the global index is found, if present.
 pub fn find_chunk_idx(chunk_ends: &Array, index: usize) -> (usize, usize) {
-    assert!(
-        index <= chunk_ends.len(),
-        "Index out of bounds of the array"
-    );
-
     // Since there might be duplicate values in offsets because of empty chunks we want to search from right
     // and take the last chunk (we subtract 1 since there's a leading 0)
     let index_chunk = search_sorted(chunk_ends, index, SearchSortedSide::Right)