On Wed, 11 Jan 2023 19:34:39 GMT, Raffaello Giulietti <rgiulie...@openjdk.org> 
wrote:

>> This change transforms a O(n^2) path to O(n) when prepending zero padding to 
>> decimal outputs, where n is the length of the padding.
>
> Raffaello Giulietti has updated the pull request incrementally with one 
> additional commit since the last revision:
> 
>   8299677: Formatter.format might take a long time to format an integer or 
> floating-point

src/java.base/share/classes/java/util/Formatter.java line 4600:

> 4598:             // apply zero padding
> 4599:             if (width > sb.length() && Flags.contains(f, 
> Flags.ZERO_PAD)) {
> 4600:                 String zeros = Character.toString(zero).repeat(width - 
> sb.length());

`Character.toString(zero)` could be replaced with `String.valueOf(zero)` saving 
one more method call

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

PR: https://git.openjdk.org/jdk/pull/11939

Reply via email to