On Sat, 30 Aug 2025 01:17:28 GMT, Bradford Wetmore <[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 35 commits: >> >> - Merge branch 'master' into 8044609-ssl >> - 1 file omitted during merge >> - Merge branch 'master' into 8044609-ssl >> - Merge branch 'master' into 8044609-ssl >> - Merge branch 'master' into 8044609-ssl >> - remove whitespace >> - erroneous edit to test file >> - remove legacy test and minor fix ups >> - Merge branch 'master' into 8044609-ssl >> - enums refactoring and line width correction >> - ... and 25 more: https://git.openjdk.org/jdk/compare/2b44ed70...71aa0211 > > src/java.base/share/classes/sun/security/ssl/SSLLogger.java line 198: > >> 196: System.err.println("all turn on all debugging"); >> 197: System.err.println("ssl turn on ssl debugging"); >> 198: System.err.println(); > > The ordering of the options seem random. Might suggest either alphabetical > or by functional area (e.g. key/trustmanager together) Ordered them in alphabetical order now. Thanks ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/18764#discussion_r2314227218
