On Mon, 16 Mar 2026 14:37:29 GMT, Artur Barashev <[email protected]> wrote:

>> src/java.base/share/classes/com/sun/crypto/provider/Argon2Impl.java line 202:
>> 
>>> 200:             this.type = type;
>>> 201:             this.lanes = parallelism;
>>> 202:             this.segLen = memory / (parallelism * 4);
>> 
>> Now when we have `ARGON2_SLICE_NUM` declared above, we can replace `4` with 
>> it.
>
> Nit: replace `parallelism` with `this.lanes` everywhere except on initial 
> assignment for consistency.

Ok.

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

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

Reply via email to