This is an automated email from the ASF dual-hosted git repository.
alamb pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/arrow-rs.git
The following commit(s) were added to refs/heads/main by this push:
new 7a85e907a4 add `#[inline]` to `BitIterator` `next` function (#9177)
7a85e907a4 is described below
commit 7a85e907a4da047ae365864aa7e15ebc750b72fb
Author: Raz Luvaton <[email protected]>
AuthorDate: Thu Jan 15 00:27:16 2026 +0200
add `#[inline]` to `BitIterator` `next` function (#9177)
# Which issue does this PR close?
N/A
# Rationale for this change
making it inlined opening opportunities for SIMD and more
# What changes are included in this PR?
just added inline attribute
# Are these changes tested?
Existing tests
# Are there any user-facing changes?
No API change
Co-authored-by: Daniƫl Heres <[email protected]>
---
arrow-buffer/src/util/bit_iterator.rs | 2 ++
1 file changed, 2 insertions(+)
diff --git a/arrow-buffer/src/util/bit_iterator.rs
b/arrow-buffer/src/util/bit_iterator.rs
index 0aa94a5d4d..f5e0811e2d 100644
--- a/arrow-buffer/src/util/bit_iterator.rs
+++ b/arrow-buffer/src/util/bit_iterator.rs
@@ -57,6 +57,7 @@ impl<'a> BitIterator<'a> {
impl Iterator for BitIterator<'_> {
type Item = bool;
+ #[inline]
fn next(&mut self) -> Option<Self::Item> {
if self.current_offset == self.end_offset {
return None;
@@ -80,6 +81,7 @@ impl Iterator for BitIterator<'_> {
self.len()
}
+ #[inline]
fn nth(&mut self, n: usize) -> Option<Self::Item> {
// Check if we can advance to the desired offset.
// When n is 0 it means we want the next() value