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