On Mon, 12 Aug 2024 15:28:47 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 ex) {            // rethrow VM errors
>> 
>> Part of me things this should be Throwable to allow for possible runtime 
>> exceptions when timed-parking virtual threads. There is a separate work 
>> needed to ensure a runtime exception is never throw but it would at least 
>> cancel here.
>
> Or use (Error | RuntimeException ex)?

That would be okay too as there are no checked exceptions here.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/20548#discussion_r1714039963

Reply via email to