RFR: 8346985: Convert test/jdk/com/sun/jdi/ClassUnloadEventTest.java to Class-File API

Chen Liang liach at openjdk.org
Fri Jan 3 16:49:40 UTC 2025


On Fri, 3 Jan 2025 13:02:07 GMT, Adam Sotona <asotona at openjdk.org> wrote:

> This patch converts the remaining test/jdk/com/sun/jdi/ClassUnloadEventTest.java from ASM to Class-File API.
> 
> Please review.
> 
> Thanks,
> Adam

Marked as reviewed by liach (Reviewer).

test/jdk/com/sun/jdi/ClassUnloadEventTest.java line 72:

> 70:             ClassWriter cw = new ClassWriter(0);
> 71: 
> 72:             cw.visit(52, ACC_SUPER | ACC_PUBLIC, name, null, "java/lang/Object", null);

This anticipates an internal name but gets a binary name passed. This works in the old code accidentally because its classes were in the unnamed package.  The port fixes this bug.

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

PR Review: https://git.openjdk.org/jdk/pull/22915#pullrequestreview-2529398187
PR Review Comment: https://git.openjdk.org/jdk/pull/22915#discussion_r1901972466


More information about the serviceability-dev mailing list