On Sun, 30 Aug 2026 06:28:49 GMT, Alan Bateman <[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.

@AlanBateman @jaikiran Thanks for the reviews and testing. Could you please 
sponsor this PR? Thanks.

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

PR Comment: https://git.openjdk.org/jdk/pull/32590#issuecomment-5487297604

Reply via email to