RFR: 8257505: nsk/share/test/StressOptions stressTime is scaled in getter but not when printed

Stefan Johansson sjohanss at openjdk.java.net
Wed Dec 2 08:56:54 UTC 2020


On Tue, 1 Dec 2020 22:45:38 GMT, Kim Barrett <kbarrett at openjdk.org> wrote:

>> Please review this small change to print the correct stress time for tests using `StressOptions`.
>> 
>> Quite recently the `getTime()` method on `StressOptions` was changed to scale the stress time using the JTREG timeout factor. See: [JDK-8252522](https://bugs.openjdk.java.net/browse/JDK-8252522)
>> 
>> The `printInfo()` method was left using the raw `time` member and will thus output a different time from the one actually used. This change fixes this by using the getter. For consistency the other values are fetched by the getters as well.
>
> Looks good, and trivial.

Thanks @kimbarrett and @dholmes-ora for the reviews.

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

PR: https://git.openjdk.java.net/jdk/pull/1545


More information about the hotspot-dev mailing list