commit 615d7830fdaf994ebd32ca1e349daf68b18c99b0
author Boaz <berman.boaz@gmail.com> Sun Sep 19 18:05:39 2021 +0300
committer GitHub <noreply@github.com> Sun Sep 19 17:05:39 2021 +0200
tree 68744e966ad8b56bf429cbafaab193e443aa0cef
parent 499301c7ff89b3dcf675bd067442077ff574eba7

Make BooleanBufferBuilder get_bit not require mutable reference (#784)

arrow/src/array/builder.rs

diff --git a/arrow/src/array/builder.rs b/arrow/src/array/builder.rs
index 8139b79..50a9319 100644
--- a/arrow/src/array/builder.rs
+++ b/arrow/src/array/builder.rs
@@ -329,7 +329,7 @@
     }
 
     #[inline]
-    pub fn get_bit(&mut self, index: usize) -> bool {
+    pub fn get_bit(&self, index: usize) -> bool {
         bit_util::get_bit(self.buffer.as_slice(), index)
     }
 
@@ -2709,6 +2709,17 @@
     }
 
     #[test]
+    fn test_bool_buffer_builder_get_first_bit_not_requires_mutability() {
+        let buffer = {
+            let mut buffer = BooleanBufferBuilder::new(16);
+            buffer.append_n(8, true);
+            buffer
+        };
+
+        assert!(buffer.get_bit(0));
+    }
+
+    #[test]
     fn test_bool_buffer_builder_get_last_bit() {
         let mut buffer = BooleanBufferBuilder::new(16);
         buffer.append_n(8, true);