On Tue, 23 Sep 2025 14:43:19 GMT, Thomas Stuefe <stu...@openjdk.org> wrote:

>> Roger Riggs has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   Simplify the code to match and remove expected environment variables on 
>> MacOSX.
>
> test/jdk/java/lang/ProcessBuilder/Basic.java line 801:
> 
>> 799:         // Check and remove TMPDIR
>> 800:         cleanedVars = matchAndRemove(cleanedVars,
>> 801:                 "TMPDIR=[^,]*,");
> 
> Why not just use reluctant match? As in `"TMPDIR=.*?,"`

Thanks, I'll take that up next time.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/27434#discussion_r2372810756

Reply via email to