RFR: 8294961: Convert java.base/java.lang.reflect.ProxyGenerator to use the Classfile API to generate proxy classes [v10]
Mandy Chung
mchung at openjdk.org
Wed Mar 22 18:56:08 UTC 2023
On Wed, 22 Mar 2023 17:37:07 GMT, Adam Sotona <asotona at openjdk.org> wrote:
>> java.base java.lang.reflect.ProxyGenerator uses ASM to generate proxy classes and this patch converts it to use Classfile API.
>>
>> Please review.
>>
>> Thank you,
>> Adam
>
> Adam Sotona has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 201 commits:
>
> - Merge branch 'master' into JDK-8294961
>
> # Conflicts:
> # src/java.base/share/classes/java/lang/reflect/ProxyGenerator.java
> - ClassEntry caching ClassDesc to improve performance
> - MethodTypeDesc performance improvements
> - improved method descriptor construction
> - Apply suggestions from code review
>
> Co-authored-by: liach <7806504+liach at users.noreply.github.com>
> - Merge branch 'master' into JDK-8294961-proxy
> - Merge branch 'master' into JDK-8294961-proxy
> - Merge branch 'JDK-8294982' into JDK-8294961
> - removed obsolete javadoc from implementation classes
> - minor fix in CodeBuilder and added test cases to LDCTest
> - ... and 191 more: https://git.openjdk.org/jdk/compare/a2d8f634...2787185c
Mapping from a Class -> String -> TypeKind can be short-circuit. The implementation of `TypeKind::from(Class<?>)` can be updated for that. The Classfile API can be considered to take a live `Class` object or `MethodType` object to avoid the conversion when appropriate.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/10991#issuecomment-1480095735
More information about the core-libs-dev
mailing list