RFR: 7903961: Make default timeout configurable [v2]

Christian Stein cstein at openjdk.org
Fri Apr 4 05:16:59 UTC 2025


On Fri, 4 Apr 2025 01:45:09 GMT, Jaikiran Pai <jpai at openjdk.org> wrote:

>> Christian Stein has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Simplify implementation, don't expose default timeout value
>
> 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.

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

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


More information about the jtreg-dev mailing list