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

Reply via email to