RFR: 8321622: ClassFile.verify(byte[] bytes) throws unexpected ConstantPoolException, IAE

Jaikiran Pai jpai at openjdk.org
Thu May 16 07:08:03 UTC 2024


On Thu, 16 May 2024 06:44:29 GMT, Adam Sotona <asotona at openjdk.org> wrote:

> Class-File API `VerifierImpl` contains debugging stack trace print causing unexpected output and confusion in specific verification cases.
> 
> This patch removes the stack trace print.
> 
> Please review.
> 
> Thanks,
> Adam

Hello Adam, the change looks OK to me. The file needs a copyright year update before integrating.

On an unrelated note, the class level javadoc has some `@see` usages which use URLs pointing to `https://raw.githubusercontent.com/openjdk/jdk...`. Are those intentional? Of course, this is an internal class so the javadoc won't be published.

-------------

Marked as reviewed by jpai (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/19258#pullrequestreview-2059775369


More information about the core-libs-dev mailing list