On Mon, 29 Jul 2024 17:18:59 GMT, Daniel Jeliński <djelin...@openjdk.org> wrote:
>> Anthony Scarpino has updated the pull request with a new target base due to >> a merge or a rebase. The pull request now contains 21 commits: >> >> - Rework TLSBase for better testing >> - Tests working >> - Merge branch 'master' into nst-multi >> - new changes >> - remove frag issue >> - Comments, remove thread, set NST default to 1, allow 0 >> - comment cleanup >> - Merge branch 'master' into nst-multi >> - copyright & cleanup >> - oops BAOS >> - ... and 11 more: https://git.openjdk.org/jdk/compare/3796fdfc...35bfe799 > > test/jdk/sun/security/ssl/SSLSessionImpl/MultiNSTParallel.java line 199: > >> 197: System.err.println("released: " + t.getName()); >> 198: } >> 199: } catch (Exception e) { > > is this catch necessary? Apparently not.. must have been an old catch ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/19465#discussion_r1700724466
