On Wed, 16 Sep 2026 21:46:25 GMT, Artur Barashev <[email protected]> wrote:

>> Hai-May Chao has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   Update MLDSANotAllowedInTLS12 test for mldsa order
>
> test/jdk/sun/security/ssl/SignatureScheme/MLDSANotAllowedInTLS12.java line 
> 119:
> 
>> 117: 
>> 118:         MLDSA_SCHEMES.forEach(signatureScheme ->
>> 119:                 assertTrue(signatureSchemes.contains(signatureScheme),
> 
> `contains()` loop is redundant: the above `assertEquals` check already proves 
> that all three schemes are present.

Removed `contains()` loop.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/32529#discussion_r4031160867

Reply via email to