RFR: 7903750: TimeBudget (-tb) does not fulfill its promisses [v5]
Jiří Vaněk
jvanek at openjdk.org
Mon Mar 24 18:18:55 UTC 2025
On Mon, 24 Mar 2025 18:01:51 GMT, Galder Zamarreño <galder at openjdk.org> wrote:
>> Jiří Vaněk has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Removed unnecesasary NPE check in getTimeBudget
>>
>> The NPE workaround was causing NPE to be only delayed for later
>
> jcstress-core/src/main/java/org/openjdk/jcstress/JCStress.java line 90:
>
>> 88: private TimeBudget getTimeBudget(ConfigsWithScheduler config) {
>> 89: TimeBudget timeBudget = new TimeBudget(config.configs.size(), opts.timeBudget());
>> 90: return timeBudget;
>
> Suggestion:
>
> return new TimeBudget(config.configs.size(), opts.timeBudget());
Sure. TY!
-------------
PR Review Comment: https://git.openjdk.org/jcstress/pull/161#discussion_r2010706445
More information about the jcstress-dev
mailing list