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