RFR: 8264029: Replace uses of StringBuffer with StringBuilder in java.base
Aleksey Shipilev
shade at openjdk.java.net
Wed Mar 24 18:14:49 UTC 2021
On Tue, 23 Mar 2021 21:54:33 GMT, Pavel Rappo <prappo at openjdk.org> wrote:
>> I'm not sure how String.repeat can be used here. Repeated String is not constant and different for each iteration.
>
> Long runs of whitespace, especially in blank strings, may have poor readability. I was thinking that
>
> " ".repeat(7) and " ".repeat(10)
>
>
> might read better than
>
> " " and " "
>
> respectively. If you don't agree, then simply disregard what I said. If you agree but worry about negative performance implications, consider creating these strings before the respective for-loops.
I suggest we ignore the use case for `String.repeat` here, and leave the patch as is.
-------------
PR: https://git.openjdk.java.net/jdk/pull/2922
More information about the nio-dev
mailing list