Skip to content

Commit ed7bbe2

Browse files
committed
Simplify zero-length list element sizing
Signed-off-by: Daniel King <dan@spiraldb.com>
1 parent dc11476 commit ed7bbe2

1 file changed

Lines changed: 4 additions & 8 deletions

File tree

  • vortex-array/src/arrays/list/compute

vortex-array/src/arrays/list/compute/take.rs

Lines changed: 4 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -402,13 +402,13 @@ where
402402
S: UnsignedPType,
403403
Offset: UnsignedPType,
404404
{
405+
if length == 0 {
406+
return Ok(0);
407+
}
408+
405409
let mut total = 0usize;
406410
for &start in starts {
407411
let start: usize = start.as_();
408-
if length == 0 {
409-
continue;
410-
}
411-
412412
let offset_range = &offsets[start..][..=length];
413413
let element_start: usize = offset_range[0].as_();
414414
let element_end: usize = offset_range[length].as_();
@@ -438,10 +438,6 @@ where
438438
for (&start, &length) in starts.iter().zip_eq(lengths) {
439439
let start: usize = start.as_();
440440
let length: usize = length.as_();
441-
if length == 0 {
442-
continue;
443-
}
444-
445441
let offset_range = &offsets[start..][..=length];
446442
let element_start: usize = offset_range[0].as_();
447443
let element_end: usize = offset_range[length].as_();

0 commit comments

Comments
 (0)