On Mon, 23 Feb 2026 23:32:50 GMT, Hai-May Chao <[email protected]> wrote:
>> Sean Coffey has updated the pull request with a new target base due to a >> merge or a rebase. The pull request now contains 11 commits: >> >> - Clean up sync of DebugPropertyValuesTest.java >> - Remove verbose option for now, difficult to exercise code path >> - Merge branch 'master' into 8371333-ssl-debug >> - Merge branch 'master' into 8371333-ssl-debug >> - Incorportate comments from Jamil >> - Merge branch 'master' into 8371333-ssl-debug >> - Merge branch 'master' into 8371333-ssl-debug >> - Move wrapText method to Utilities >> - Merge branch 'master' into 8371333-ssl-debug >> - use LINE_SEP >> - ... and 1 more: https://git.openjdk.org/jdk/compare/a2961293...ed4c5687 > > test/jdk/sun/security/ssl/SSLLogger/DebugPropertyValuesTest.java line 64: > >> 62: List.of("\"logger\".*: \"javax.net.ssl\",", >> 63: "\"specifics\" : \\[", >> 64: "\"message\".*: \"Produced ClientHello handshake >> message")); > > It looks like we could keep the original code as is with 8-space indent. Hi @haimaychao - line 64 was extending past the recommended 80 char limit. That's something that Brad likes to keep in check so I pulled these lines in a bit. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/28511#discussion_r2846596666
