On Fri, 15 Apr 2022 21:15:45 GMT, Paul Sandoz <[email protected]> wrote:
>> Alan Bateman has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Refresh
>
> src/java.base/share/classes/jdk/internal/vm/Continuation.java line 94:
>
>> 92: default:
>> 93: throw new AssertionError("Unknown pinned reason: " +
>> reason);
>> 94: }
>
> Suggestion:
>
> return switch (reason) {
> case 2 -> Pinned.CRITICAL_SECTION;
> case 3 -> Pinned.NATIVE;
> case 4 -> Pinned.MONITOR;
> default -> throw new AssertionError("Unknown pinned reason: " +
> reason);
> };
That would be clearer (I think this predates switch expressions).
-------------
PR: https://git.openjdk.java.net/jdk/pull/8166
