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