On Mon, 27 Jun 2022 04:50:09 GMT, Thejasvi Voniadka <tvonia...@openjdk.org> wrote:
> This is a simple change to add some protective code to > java/util/zip/ZipFile/DeleteTempJar.java so that the clean-up is more > complete in the event of encountering an any exceptions while running. In the > present state, the server.stop() does not get executed in case of > encountering a failure. As a result, the test may get hung until a timeout. > This fix should address the scenario. > > The test passes on all supported platforms post fix. test/jdk/java/util/zip/ZipFile/DeleteTempJar.java line 89: > 87: server.stop(0); > 88: } > 89: } Is there a reason to put the server.start in the try block, I would think this is simpler: server.start(); try { : } finally { server.stop(0); } ------------- PR: https://git.openjdk.org/jdk/pull/9293