perf(array): avoid double bounds check in ArrayIterator (risingwavela…

…bs#8685) Signed-off-by: TennyZhuang <zty0826@gmail.com>
Little-Wallace · Mar 21, 2023 · 0e8f518 · 0e8f518
1 parent 16c9708
commit 0e8f518
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/src/common/src/array/iterator.rs b/src/common/src/array/iterator.rs
@@ -34,7 +34,8 @@ impl<'a, A: Array> Iterator for ArrayIterator<'a, A> {
         if self.pos >= self.data.len() {
             None
         } else {
-            let item = self.data.value_at(self.pos);
+            // SAFETY: bounds check is done by `self.pos < self.data.len()`.
+            let item = unsafe { self.data.value_at_unchecked(self.pos) };
             self.pos += 1;
             Some(item)
         }