On Thu, 22 Feb 2024 12:52:19 GMT, Matthew Donovan <mdono...@openjdk.org> wrote:

>> In this PR I updated the tests to use the newer 
>> ProcessTools.createTestJavaProcessBuilder() methods to launch child 
>> processes. This method ensures that VM options are passed to subprocesses.
>
> Matthew Donovan has updated the pull request incrementally with one 
> additional commit since the last revision:
> 
>   reverted Compatibility.java to avoid possible conflicts with VM options and 
> older JDKs.

Nit: Copyright year of IterationCount.java would need to be changed.

Looks good. Thanks.

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

Marked as reviewed by hchao (Committer).

PR Review: https://git.openjdk.org/jdk/pull/17877#pullrequestreview-1901649894
PR Comment: https://git.openjdk.org/jdk/pull/17877#issuecomment-1964804507

Reply via email to