On Mon, 12 Aug 2024 21:37:45 GMT, Doug Lea <d...@openjdk.org> wrote:
>> src/java.base/share/classes/java/util/concurrent/locks/AbstractQueuedLongSynchronizer.java
>> line 381:
>>
>>> 379: else
>>> 380: break;
>>> 381: } catch (Error | RuntimeException ex) {
>>
>> @DougLea @AlanBateman Changed to Error | RuntimeException here and for AQS
>
> lgtm!
`catch (Throwable ex)` would be consistent with the similar block at line 331.
Though I'm unclear how that compiles without the method declaring `throws
Throwable` ??
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/20548#discussion_r1714470560
