On Wed, 19 Nov 2025 15:48:49 GMT, Weijun Wang <[email protected]> wrote:

> Looks good.
> 
> I assume except for `SSLLogger` itself and the test change, everything else 
> is just `s/isOn/isOn()/`, right? I quickly paged down and see no other 
> significant changes.

Thanks for the review. Yes, most changes are `s/isOn/isOn()/`

1 line addition to test case like Daniel noted. Useful to confirm that change 
is indeed having effect

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

PR Comment: https://git.openjdk.org/jdk/pull/28376#issuecomment-3553509576

Reply via email to