On Tue, 13 Oct 2026 17:38:56 GMT, Matthew Donovan <[email protected]> wrote:

>> Mikhail Yankelevich has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   Change capital S to s
>
> test/jdk/sun/security/provider/KeyStore/DKSSystemPropertiesTest.java line 92:
> 
>> 90:             if (!keystore.getCreationDate(alias).equals(
>> 91:                     Date.from(keystore.getCreationInstant(alias)))
>> 92:             ) {
> 
> kind of strange to put this on a new line

It's quite hard for me personally to detact may `)` next to each other and I 
know I'm not the only one. So I'm doing this so it's always easy to read at a 
glance

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

PR Review Comment: https://git.openjdk.org/jdk/pull/30712#discussion_r4383040200

Reply via email to