alamb opened a new issue #996:
URL: https://github.com/apache/arrow-rs/issues/996


   **Is your feature request related to a problem or challenge? Please describe 
what you are trying to do.**
   As pointed out by @jhorstmann  on #921 at 
https://github.com/apache/arrow-rs/pull/921#discussion_r760086317, 
   
   `Buffer::typed_data` does not actually need to be marked unsafe since it 
checks the alignment requirements. The very similarly implemented 
`MutableBuffer::typed_data_mut` is not marked as unsafe. The safety notes 
mention bool as a special case, but that is no longer an ArrowNativeType since 
a while.
   
   
https://github.com/apache/arrow-rs/blob/6a6e7f7/arrow/src/buffer/immutable.rs#L160-L181
   
   
   **Describe the solution you'd like**
   1. Remove `unsafe` from `Buffer::typed_data`
   2. Remove note about bool in docstrings


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to