On Mon, 13 Mar 2023 21:02:51 GMT, Adam Sotona <asot...@openjdk.org> wrote:

>> jdk/classfile/CorpusTest.java rarely fails in 1 of 6754 parametrised 
>> parallel junit tests
>> 
>> The root cause seems to be thread-unsafe lazy initialisation of 
>> `TypeKind.newarraycodeToTypeTag` in `TypeKind::fromNewArrayCode`. Provided 
>> patch replaces that lazy-initialized helper array with switch expression.
>> 
>> Please review.
>> 
>> Thanks,
>> Adam
>
> Adam Sotona has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   throwing IllegalArgumentException instead of IllegalStateException

Looks OK to me.

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

Marked as reviewed by jpai (Reviewer).

PR: https://git.openjdk.org/jdk/pull/13004

Reply via email to