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 changes to `ProcessTools.java` looks good to me. I haven't paid close 
attention to the other changes but they are the same as what was previously 
integrated through JDK-8387729.

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

Marked as reviewed by jpai (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/32590#pullrequestreview-5061054860

Reply via email to