RFR: 8286474: Drop --enable-preview from Sealed Classes related tests

Alan Bateman alanb at openjdk.java.net
Tue May 10 12:51:07 UTC 2022


On Tue, 10 May 2022 12:07:31 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:

> There are plenty of tests failing on many architectures due to `--enable-preview` VM code introduced by Loom. This improvement eliminates some of the redundant `--enable-preview` clauses from the Sealed Classes tests, since Sealed Classes have been graduated from preview in JDK 17.
> 
> Additional testing:
>  - [x] Linux x86_64 fastdebug, affected tests still pass
>  - [x] Linux x86_32 fastdebug, affected tests start to pass

test/jdk/java/lang/reflect/sealed_classes/SealedClassesReflectionTest.java line 29:

> 27:  * @summary reflection test for sealed classes
> 28:  * @compile -source ${jdk.version} SealedClassesReflectionTest.java
> 29:  * @run testng/othervm SealedClassesReflectionTest

You should be able to drop` -source ${jdk.version}` too. It was required when compiling with `--enable-preview`.

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

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


More information about the core-libs-dev mailing list