On Thu, 21 Dec 2023 07:55:50 GMT, Jaikiran Pai <j...@openjdk.org> wrote:
>> Jim Laskey has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Clear sooner > > test/jdk/java/lang/StringBuilder/StringBufferRepeat.java line 138: > >> 136: sb.repeat('*', 5); >> 137: expected = "*****"; >> 138: assertEquals(expected, sb.toString()); > > Hello Jim, just a minor detail - in case of testng's `Assert.assertEquals()` > the first param is the `actual` and the second is the `expected`. Some other > existing usages of this method in this test also have the incorrect order. Updated ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/17172#discussion_r1441707877