On Wed, 28 Feb 2024 00:00:01 GMT, Chad Rakoczy <d...@openjdk.org> wrote:
>> [JDK-8299677](https://bugs.openjdk.java.net/browse/JDK-8299677) fixes a bug >> with Formatter.format taking a long time when there is a lot of padding. >> This test runs Formatter.format with very large padding. Test fails before >> [JDK-8299677](https://bugs.openjdk.java.net/browse/JDK-8299677) and passes >> after. >> >> Timeout for the test was set to 10 seconds. Test passes locally with as low >> as 1 (after [JDK-8299677](https://bugs.openjdk.java.net/browse/JDK-8299677)) >> and fails as high as 120 (before >> [JDK-8299677](https://bugs.openjdk.java.net/browse/JDK-8299677)) so it >> should be consistent. > > Chad Rakoczy has updated the pull request incrementally with one additional > commit since the last revision: > > Test updates test/jdk/java/util/Formatter/Padding.java line 44: > 42: > 43: private static final String tenMillionZeros = "0".repeat(10000000); > 44: private static final String tenMillionBlanks = " ".repeat(10000000); Just a nit to help readability Suggestion: private static final String tenMillionZeros = "0".repeat(10_000_000); private static final String tenMillionBlanks = " ".repeat(10_000_000); ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/18033#discussion_r1509251657