On Mon, 26 Oct 2026 15:54:35 GMT, Artur Barashev <[email protected]> wrote:

>> Thejasvi Voniadka has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   Updated copyrights, added comments, and moved the test to the right 
>> directory
>
> test/jdk/sun/security/ssl/TrustStoreManager/StaleTrustedCerts.java line 105:
> 
>> 103: 
>> 104:         assertEquals(2, 
>> defaultTrustManager().getAcceptedIssuers().length);
>> 105:         Reference.reachabilityFence(first);
> 
> Please place a few comment throughout the `main()` explaining what we are 
> doing at each step.

Done, updated the comments, and also moved the test to an existing directory to 
logically group it together with existing tests.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/33545#discussion_r4390794509

Reply via email to