On Fri, 17 Feb 2023 15:17:59 GMT, Roger Riggs <rri...@openjdk.org> wrote:
>> Matthew Donovan has updated the pull request with a new target base due to a >> merge or a rebase. The incremental webrev excludes the unrelated changes >> brought in by the merge/rebase. The pull request contains six additional >> commits since the last revision: >> >> - added System.exit when exceptions are thrown and refactored for clarity >> - Merge branch 'master' into rmi-sslparams >> - added default switch case and additional refactoring >> - added exceptions for cases 4 and 5 >> - clarified cases 4 and 5 >> - 8298939: Refactor open/test/jdk/javax/rmi/ssl/SSLSocketParametersTest.sh >> to jtreg java test > > test/jdk/javax/rmi/ssl/SSLSocketParametersTest.java line 261: > >> 259: } >> 260: break; >> 261: default: > > The default should be retained to produce an error. I put the default case back ------------- PR: https://git.openjdk.org/jdk/pull/11910
