On Wed, 12 May 2021 22:30:30 GMT, Mandy Chung <mch...@openjdk.org> wrote:
>> Harold Seigel has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> test changes and small fixes
>
> src/hotspot/share/classfile/classLoaderData.cpp line 299:
>
>> 297: }
>> 298:
>> 299: // Weak hidden classes have their own ClassLoaderData that is marked to
>> keep alive
>
> `s/Weak/Non-strong/`
fixed. thanks for finding it.
> test/hotspot/jtreg/runtime/HiddenClasses/StressHiddenClasses.java line 39:
>
>> 37: import jdk.test.lib.compiler.InMemoryJavaCompiler;
>> 38:
>> 39: public class StressHiddenClasses {
>
> Since `StressClassLoadingTest` is revised to test hidden class, this test is
> a subset of it.
> I think this can be removed as JDK-8265694 suggests.
Thanks Mandy. I will remove the test as the fix for JDK-8265694 after this
change is pushed.
-------------
PR: https://git.openjdk.java.net/jdk/pull/3974
