On Thu, 20 Aug 2026 14:08:18 GMT, Sean Mullan <[email protected]> wrote:

>> Valerie Peng has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   import cleanup.
>
> src/java.base/share/classes/com/sun/crypto/provider/Argon2DerivedKey.java 
> line 60:
> 
>> 58:     // for including Argon2 parameters in toString() method
>> 59:     private final transient String info;
>> 60:     private transient Cleaner.Cleanable cleaner;
> 
> Probably not necessary to mark these `transient` now that serialization is 
> not supported.

I still need the transient on `cleaner` in order to avoid the warning of an 
non-transient field in a serializable class declared with a non-serializable 
type.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/29597#discussion_r4203906959

Reply via email to