On Thu, 13 Aug 2026 11:31:06 GMT, Alan Bateman <[email protected]> wrote:

>> Thanks for the suggestion. I used `ProcessTools.executeTestJava()` because 
>> `@run main/othervm --finalization enabled FinalizationOption yes` would 
>> cause jtreg to treat `enabled` as the test class, so the 
>> whitespace-separated form needs to be launched explicitly here.
>
> The changes means that FinalizationOption is run directly by jtreg and 
> launched by the test itself.
> 
> Brent - what would you think about always launching, e.g.
> 
> 
> @run main FinalizationOption disabled form1
> @run main FinalizationOption disabled form2
> @run main FinalizationOption enabled form1
> @run main FinalizationOption enabled form2
> 
> or some variation.

Meaning that the test would always be launched via ProcessTools? Sure, that 
seems fine. I think it's better to have all the test variants be launched in 
the same way, rather than the current mixture.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/31928#discussion_r3799919291

Reply via email to