On Thu, 14 Jul 2022 11:57:37 GMT, Doug Lea <d...@openjdk.org> 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