RFR: 8372004: Have SSLLogger implement System.Logger [v3]
Sean Coffey
coffeys at openjdk.org
Wed Nov 19 16:07:34 UTC 2025
On Wed, 19 Nov 2025 15:48:49 GMT, Weijun Wang <weijun at openjdk.org> 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
More information about the security-dev
mailing list