RFR: 7903961: Make default timeout configurable [v2]

Jaikiran Pai jpai at openjdk.org
Fri Apr 4 06:32:02 UTC 2025


On Fri, 4 Apr 2025 05:14:41 GMT, Christian Stein <cstein at openjdk.org> wrote:

>> src/share/classes/com/sun/javatest/regtest/exec/RegressionScript.java line 1351:
>> 
>>> 1349:     private final TestProperties properties;
>>> 1350: 
>>> 1351:     public RegressionScript(TestProperties properties) {
>> 
>> I did not notice this in the previous round, sorry, but this constructor can be `private` like the other one that previously was.
>
> No, it has to remain `public` because it is used by `RegressionTestSuite` in the `config` package.

You are right, I missed that.

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

PR Review Comment: https://git.openjdk.org/jtreg/pull/253#discussion_r2028169488


More information about the jtreg-dev mailing list