This is an automated email from the ASF dual-hosted git repository.

alamb pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/arrow-rs.git


The following commit(s) were added to refs/heads/master by this push:
     new 239cba1  Remove out of date comment (#1008)
239cba1 is described below

commit 239cba141cb27519b7c32d58a3ea6447fda31d11
Author: Andrew Lamb <[email protected]>
AuthorDate: Mon Dec 6 15:39:52 2021 -0500

    Remove out of date comment (#1008)
---
 arrow/src/buffer/immutable.rs | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/arrow/src/buffer/immutable.rs b/arrow/src/buffer/immutable.rs
index 5ba2548..bd09702 100644
--- a/arrow/src/buffer/immutable.rs
+++ b/arrow/src/buffer/immutable.rs
@@ -164,10 +164,6 @@ impl Buffer {
     /// `ArrowNativeType` is public so that it can be used as a trait bound 
for other public
     /// components, such as the `ToByteSlice` trait.  However, this means that 
it can be
     /// implemented by user defined types, which it is not intended for.
-    ///
-    /// Also `typed_data::<bool>` is unsafe as `0x00` and `0x01` are the only 
valid values for
-    /// `bool` in Rust.  However, `bool` arrays in Arrow are bit-packed which 
breaks this condition.
-    /// View buffer as typed slice.
     pub unsafe fn typed_data<T: ArrowNativeType + num::Num>(&self) -> &[T] {
         // JUSTIFICATION
         //  Benefit

Reply via email to