RFR: 8260576: Typo in compiler/runtime/safepoints/TestRegisterRestoring.java

Tobias Hartmann thartmann at openjdk.java.net
Mon Feb 1 18:46:53 UTC 2021


On Thu, 28 Jan 2021 11:23:22 GMT, xpbob <github.com+7837910+xpbob at openjdk.org> wrote:

> 8260576: Typo in compiler/runtime/safepoints/TestRegisterRestoring.java

Changes requested by thartmann (Reviewer).

Marked as reviewed by thartmann (Reviewer).

test/hotspot/jtreg/compiler/runtime/safepoints/TestRegisterRestoring.java line 50:

> 48:             for (int i = 0; i < array.length; i++) {
> 49:                 if (array[i] != 10_000) {
> 50:                     throw new RuntimeException("Test failed: array[" + i + "] = " + array[i] + " but should be 10,000");

`.` is simply the (German) thousands separator. Not sure it's really worth it changing it to a `,`. But given the exception message is in English, looks good to me.

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

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


More information about the hotspot-compiler-dev mailing list