On Mon, 30 Nov 2020 21:36:40 GMT, Igor Ignatyev <iignat...@openjdk.org> wrote:
>> @iignatev could you please review? Thank you. >> >> note to self: >> jtreg test/jdk/java/lang/ProcessBuilder/InheritIO/InheritIoTest.java >> test/jdk/java/lang/SecurityManager/modules/CustomSecurityManagerTest.java >> test/jdk/java/lang/Thread/uncaughtexceptions/UncaughtExceptionsTest.java >> test/jdk/java/lang/annotation/loaderLeak/LoaderLeakTest.java > > test/jdk/java/lang/annotation/loaderLeak/LoaderLeakTest.java line 53: > >> 51: Files.createDirectories(REPOSITORY_PATH); >> 52: List<String> classes = List.of("A.class", "B.class", "C.class"); >> 53: for (String fileName : classes) { > > is this really needed for the test to operate correctly? or can we just use > _regular_ `TEST_CLASSES` as CP? CLASSES_PATH is initialized with value of TEST_CLASSES property. It is obviously confusing variable naming so in the [new PR](https://github.com/openjdk/jdk/pull/1577) it is renamed ------------- PR: https://git.openjdk.java.net/jdk/pull/1484