RFR: 8342206: Convenience method to check if a constant pool entry matches nominal descriptors [v3]
Chen Liang
liach at openjdk.org
Fri Apr 25 16:36:53 UTC 2025
> The ClassFile API promotes usage of constant nominal descriptors over literal strings. However, for `class` file parsing, currently the efficient way to check if a constant pool entry matches a desired descriptor is via unrolling them back to strings. However, string unrolling is ugly, and often times confusing, such as internal names versus field descriptors.
>
> As a result, I propose to provide new methods that compare constant pool entries with the potential symbolic descriptors they represent. This is no less efficient as checking raw string equality, avoids exceptional failure behaviors of conversion to symbolic descriptors, and avoids potential programmer errors stemming from raw string handling. See the CSR for a full story.
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 five additional commits since the last revision:
- Extract the actual tests, move pool entry spawning to the bottom
- Merge branch 'master' of https://github.com/openjdk/jdk into feature/cf-equals-sym
- Rename UTF8 test methods to be different from other test methods
- Merge branch 'master' of https://github.com/openjdk/jdk into feature/cf-equals-sym
- 8342206: Convenience method to check if a constant pool entry matches nominal descriptors
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/23548/files
- new: https://git.openjdk.org/jdk/pull/23548/files/dd3adbc9..1f3cc936
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=23548&range=02
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=23548&range=01-02
Stats: 14467 lines in 469 files changed: 10053 ins; 3405 del; 1009 mod
Patch: https://git.openjdk.org/jdk/pull/23548.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/23548/head:pull/23548
PR: https://git.openjdk.org/jdk/pull/23548
More information about the core-libs-dev
mailing list