RFR: 8335060: ClassCastException after JDK-8294960
Jaikiran Pai
jpai at openjdk.org
Mon Jul 1 11:18:17 UTC 2024
On Wed, 26 Jun 2024 06:53:28 GMT, Adam Sotona <asotona at openjdk.org> wrote:
> Conversion of `java.lang.invoke` package to Class-File API is failing to execute method handles with specific type conversion requirements. Root cause is in the new `TypeConvertingMethodAdapter::primitiveTypeKindFromClass` implementation. Original code has been matching the types by hash code and it mistakenly appeared to be matching the primitive types.
>
> This patch fixes `TypeConvertingMethodAdapter::primitiveTypeKindFromClass` and adds tests to better cover `TypeConvertingMethodAdapter` functionality.
>
> Please review.
>
> Thanks,
> Adam
Hello Adam, I'm not familiar with this code. But looking at the call sites of this method and the code in and around those call sites (including code comments), this change looks OK to me.
-------------
Marked as reviewed by jpai (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/19898#pullrequestreview-2151108992
More information about the core-libs-dev
mailing list