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