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