On Mon, 22 Apr 2024 11:06:57 GMT, Christoph Langer <clan...@openjdk.org> wrote:
> While working in that area I found some potential for cleanup of a few tests. > > Most notably: > > B5045306.java: > - does not need to run in othervm > - the executor service that it uses should be shut down eventually to free > resources > > B8291637.java: > - use just one instead of two test VM invocations > KeepAliveTimerThread.java: > call to grp.destroy() at the end is pointless (API is void & deprecated for > removal) > > Generally: > The deprecated URL constructor is used. It can be switched to the handy > URIBuilder > Some more try with resources here and there This pull request has now been integrated. Changeset: a5005c87 Author: Christoph Langer <clan...@openjdk.org> URL: https://git.openjdk.org/jdk/commit/a5005c87c4d5598eb54e9824105767d833f9660b Stats: 290 lines in 5 files changed: 113 ins; 69 del; 108 mod 8330814: Cleanups for KeepAliveCache tests Reviewed-by: jpai, dfuchs ------------- PR: https://git.openjdk.org/jdk/pull/18884
