RFR: 8356022: Migrate descriptor parsing from generics to BytecodeDescriptor [v2]
Chen Liang
liach at openjdk.org
Wed Aug 27 19:18:17 UTC 2025
> As another step toward the removal of the old generics infrastructure, I propose to remove the usages of generic parsing utilities and use the facilities provided by BytecodeDescriptor, already used by MethodType.fromDescriptorString. This also prevents extra validation cost in use sites to defend against generic types.
>
> In this patch, BytecodeDescriptor and Wrapper see minor updates, mainly for better exception messages - previously, an unparseable char in the descriptor string just reports that char, and now the whole descriptor string is reported.
>
> These behaviors are already covered by the tests added in JDK-8350704 #23788.
>
> Testing: reflect/annotation/Class, running tier 1+2
Chen Liang has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains four additional commits since the last revision:
- Merge branch 'master' of https://github.com/openjdk/jdk into fix/generic-desc-parse
- Roll back wrapper, test for BytecodeDescriptor
- Merge branch 'master' of https://github.com/openjdk/jdk into fix/generic-desc-parse
- 8356022: Migrate descriptor parsing from generics to BytecodeDescriptor
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/24978/files
- new: https://git.openjdk.org/jdk/pull/24978/files/a0821e97..59d27edc
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=24978&range=01
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=24978&range=00-01
Stats: 328393 lines in 5954 files changed: 195916 ins; 90202 del; 42275 mod
Patch: https://git.openjdk.org/jdk/pull/24978.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/24978/head:pull/24978
PR: https://git.openjdk.org/jdk/pull/24978
More information about the core-libs-dev
mailing list