On Wed, 27 Aug 2025 18:09:55 GMT, Matthew Donovan <mdono...@openjdk.org> wrote:

> This PR extends security tests to use ByteBuffers backed by MemorySegments. 
> Tests in the areas of Signature, Cipher, MessageDigest, and Mac are updated.

test/jdk/javax/crypto/CipherSpi/CipherByteBufferOverwriteTest.java line 127:

> 125:             System.out.println("All Tests Passed");
> 126:         } finally {
> 127:             arena.close();

Would it make sense to initialize `arena` in a try-with-resources, so you can 
get rid of the explicit `.close()`?

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/26967#discussion_r2305147248

Reply via email to