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

Kim Barrett kbarrett at openjdk.java.net
Tue Dec 1 22:47:59 UTC 2020


On Tue, 1 Dec 2020 21:22:18 GMT, Stefan Johansson <sjohanss 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.

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

Marked as reviewed by kbarrett (Reviewer).

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


More information about the hotspot-dev mailing list