On Sat, 29 Aug 2026 00:23:20 GMT, Guanqiang Han <[email protected]> wrote:
> Please review this "REDO" change. Thanks! > > **Description:** > > JDK-8387729(PR #31928) added support for whitespace-separated values for > several long options. The change was backed out after some tests failed with > `JTREG_TEST_THREAD_FACTORY=Virtual`. > > **Solution:** > > Add the affected long options to `doubleWordArgs` so that `ProcessTools` > correctly handles their whitespace-separated values when > `JTREG_TEST_THREAD_FACTORY=Virtual` is used. > > **Test:** > > - GHA > - Tier1 with `JTREG_TEST_THREAD_FACTORY=Virtual` > - The following tests with and without `JTREG_TEST_THREAD_FACTORY=Virtual`: > - `test/jdk/java/foreign/enablenativeaccess/TestEnableNativeAccess.java` > - `test/jdk/java/lang/Object/FinalizationOption.java` > - `test/jdk/java/lang/Object/InvalidFinalizationOption.java` > - `test/jdk/java/lang/reflect/Field/mutateFinals/cli/CommandLineTest.java` > - `test/jdk/sun/misc/UnsafeMemoryAccessWarnings.java` > > > > --------- > - [x] I confirm that I make this contribution in accordance with the [OpenJDK > Interim AI Policy](https://openjdk.org/legal/ai). The src + test changes are identical to before. The additional update to doubleWordArgs to ProcessTools. addTestThreadFactoryArgs to allow for JTREG_TEST_THREAD_FACTORY=Virtual runs in tier5 looks right too. ------------- Marked as reviewed by alanb (Reviewer). PR Review: https://git.openjdk.org/jdk/pull/32590#pullrequestreview-5060120162
