On Fri, 18 Nov 2022 21:29:50 GMT, Joe Darcy <da...@openjdk.org> wrote:

>> `@enablePreview` wasn't available when most of these tests were initially 
>> created. I changed some of to use this tag as part of other changes but 
>> didn't cover over the remaining tests that still use `@compile 
>> --enable-preview -source ${jdk.version} ...` and `@run main/othervm 
>> --enable-preview ...`.   I just scanned the tests in 
>> test/jdk/java/lang/Thread/virtual and they can all be changed the same way, 
>> if you want. If you leave it then we'll just change them at the next edit in 
>> this area.
>
> I think I got most of those in the PR. There was a file or two I didn't 
> update because not all the @compile/@run directives in the file used 
> --enablePreview and the @enablePreview directive applies to all @compile and 
> @run directives.

Most of the tests in test/jdk/java/lang/Thread/virtual/ still have `@compile` 
and othervm, is it possible you didn't push the update with those changes?

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

PR: https://git.openjdk.org/jdk/pull/11222

Reply via email to