Rich-T-kid opened a new issue, #10350:
URL: https://github.com/apache/arrow-rs/issues/10350

   ### Is your feature request related to a problem or challenge?
   
   `round_upto_multiple_of_64` is custom, hand-rolled code that duplicates 
something the standard library now provides natively 
(`checked_next_multiple_of`)
   
   ### Describe the solution you'd like
   
   `checked_next_multiple_of` (stabilized in std) provides similar 
functionality to our internal `round_upto_multiple_of_64`, but with overflow 
safety built in via its Option return type.
   Worth investigating whether we can deprecate `round_upto_multiple_of_64` and 
replace its call sites with `checked_next_multiple_of` instead
   
   ### Describe alternatives you've considered
   
   sticking with `round_upto_multiple_of_64` but this could lead to overflows
   
   ### Additional context
   
   source : https://github.com/apache/arrow-rs/pull/10317/changes#r3589465342


-- 
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