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

>> Initial commit containing the public API changes and related regression 
>> tests.
>> 
>> ---------
>> - [x] I confirm that I make this contribution in accordance with the 
>> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai).
>
> 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.

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

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

Reply via email to