RFR: 8332614: Type-checked ConstantPool.entryByIndex and ClassReader.readEntryOrNull [v4]
Adam Sotona
asotona at openjdk.org
Wed May 29 06:45:03 UTC 2024
On Wed, 29 May 2024 05:19:28 GMT, Chen Liang <liach at openjdk.org> wrote:
>> I propose to add type-checked ConstantPool.entryByIndex and ClassReader.readEntryOrNull taking an extra Class parameter, which throws ConstantPoolException instead of ClassCastException on type mismatch, which can happen to malformed ClassFiles.
>>
>> Requesting a review from @asotona. Thanks!
>>
>> Proposal on mailing list: https://mail.openjdk.org/pipermail/classfile-api-dev/2024-May/000512.html
>
> 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 six additional commits since the last revision:
>
> - Use switch
> - Merge branch 'master' of https://github.com/openjdk/jdk into feature/entry-by-type
> - Merge branch 'master' of https://github.com/openjdk/jdk into feature/entry-by-type
> - Merge branch 'master' of https://github.com/openjdk/jdk into feature/entry-by-type
> - Use constants, beautify code
> - 8332614: Type-checked ConstantPool.entryByIndex and ClassReader.readEntryOrNull
Looks good to me, thanks!
-------------
Marked as reviewed by asotona (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/19330#pullrequestreview-2084437069
More information about the core-libs-dev
mailing list