Rafferty97 opened a new pull request, #9593:
URL: https://github.com/apache/arrow-rs/pull/9593

   # Which issue does this PR close?
   
   In response to a comment on #9494, I have searched for occurances of 
`truncate(0)` and replaced them with `clear` to be more idiomatic. I know it's 
only a minor stylistic change, but it does help with code readability.
   
   In the process, I also noticed and fixed an inconsistency between 
`MutableBuffer::truncate` and `MutableBuffer::clear`. Not sure if this is a 
latent bug or actually benign.
   
   # Rationale for this change
   
   Explained above
   
   # What changes are included in this PR?
   
   * Replace a few instances of `truncate(0)` with `clear`
   * Fix `MutableBuffer::clear` to be consistent with `MutableBuffer::truncate`
   
   # Are these changes tested?
   
   Yes, the existing test suite passes.
   
   # Are there any user-facing changes?
   
   No


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