RFR: 8243287: Removal of Unsafe::defineAnonymousClass [v3]

Harold Seigel hseigel at openjdk.java.net
Tue May 11 14:13:51 UTC 2021


On Tue, 11 May 2021 13:41:53 GMT, Alan Bateman <alanb at openjdk.org> wrote:

>> test/jdk/java/lang/Class/GetModuleTest.java line 42:
>> 
>>> 40: import static org.testng.Assert.*;
>>> 41: 
>>> 42: public class GetModuleTest {
>> 
>> testGetModuleOnVMAnonymousClass is the only test here that uses ASM so you can remove the imports as part of the removal.
>
> Can you check test/jdkjava/lang/Class/attributes/ClassAttributesTest.java? It may minimally need a comment to be updated. That's the only additional test that I could find in test/jdk.

Hi Alan,
Thanks for find this.  I removed the unneeded imports and @modules from GetModulesTest.java and updated the comment in ClassAttributesTest.java.
Thanks, Harold

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

PR: https://git.openjdk.java.net/jdk/pull/3974


More information about the serviceability-dev mailing list