On Tue, 28 Mar 2023 23:47:02 GMT, Leonid Mesnik <[email protected]> wrote:
>> Alan Bateman has updated the pull request with a new target base due to a
>> merge or a rebase. The incremental webrev excludes the unrelated changes
>> brought in by the merge/rebase. The pull request contains four additional
>> commits since the last revision:
>>
>> - ThreadSleepEvent refactoring
>> - Merge
>> - Merge
>> - Initial sync from fibers branch
>
> src/java.base/share/classes/java/lang/System.java line 2566:
>
>> 2564:
>> 2565: public <V> V executeOnCarrierThread(Callable<V> task)
>> throws Exception {
>> 2566: if (Thread.currentThread() instanceof VirtualThread
>> vthread) {
>
> Any specific reason to don't use Thread.currentThread().isVirtual()?
To use the pattern variable to call `executeOnCarrierThread` on it?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/13203#discussion_r1151270089
