On Fri, 31 May 2024 19:58:34 GMT, Chris Plummer <cjplum...@openjdk.org> wrote:
>> Leonid Mesnik has updated the pull request incrementally with one additional >> commit since the last revision: >> >> moved return out of try/catch > > test/hotspot/jtreg/vmTestbase/nsk/share/aod/DummyTargetApplication.java line > 68: > >> 66: if ((signal == null) || >> !signal.equals(AODTestRunner.SIGNAL_FINISH)) >> 67: throw new TestBug("Unexpected signal: '" + signal + "'"); >> 68: pipe.close(); > > Exceptions can be thrown before you get here. Thanks, updated. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/19505#discussion_r1623281376