On Thu, 14 Jul 2022 11:57:37 GMT, Doug Lea <[email protected]> wrote:
> This test now conforms to jtreg rules about not using System.exit to cover
> untested OutOfMemoryErrors
Hi Doug,
One pre-existing style nit but otherwise the termination logic seems okay.
Thanks.
test/jdk/java/util/concurrent/locks/Lock/OOMEInAQS.java line 108:
> 106: cond.signalAll();
> 107: } finally {
> 108: lock.unlock();
Pre-existing but the lock() should be before the try block (even if now it
can't throw exceptions).
-------------
Marked as reviewed by dholmes (Reviewer).
PR: https://git.openjdk.org/jdk/pull/9491
