On Tue, 18 Nov 2025 18:40:12 GMT, Valerie Peng <[email protected]> wrote:
>> Martin Doerr has updated the pull request incrementally with one additional >> commit since the last revision: >> >> More minor cleanup. > > src/java.base/share/classes/com/sun/crypto/provider/AES_Crypt.java line 62: > >> 60: private int[] sessionKe = null; // key for encryption >> 61: private int[] sessionKd = null; // preprocessed key for decryption >> 62: private int[] K = null; // preprocessed key in case of decryption > > I find the comment confusing as `K` is sometimes assigned with `sessionKe`, > so it can't be used only for decryption? Thanks for looking at it! I've merged my additional proposal. `K` is removed, now. Does the Java part look ok? I'll ask for a hotspot review once the Java part is fine. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/28299#discussion_r2539714325
