adamreeve opened a new pull request, #43195:
URL: https://github.com/apache/arrow/pull/43195

   ### Rationale for this change
   
   See #43142. This is a follow up to #43071 which refactored the Decryptor API 
and added extra checks to prevent segfaults. This PR makes similar changes to 
the Encryptor API for consistency and better maintainability.
   
   ### What changes are included in this PR?
   
   * Change `AesEncryptor::Encrypt` and `Encryptor::Encrypt` to use 
`arrow::util::span` instead of raw pointers
   * Replace the `AesEncryptor::CiphertextSizeDelta` method with a 
`CiphertextLength` method that checks for overflow and abstracts the size 
difference behaviour away from consumer code for improved readability.
   
   ### Are these changes tested?
   
   * This is mostly a refactoring of existing code so is covered by existing 
tests.
   
   ### 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