<i18n dev> RFR: 8294397: Replace StringBuffer with StringBuilder within java.text [v4]

Jonathan Gibbons jjg at openjdk.org
Mon Oct 3 20:46:17 UTC 2022


On Fri, 30 Sep 2022 20:08:10 GMT, Justin Lu <duke at openjdk.org> wrote:

>> Problem: Unnecessary instances of StringBuffer within java.text (internal only)
>> 
>> Fix: StringBuffer Replaced with StringBuilder, and adjusted variable/method names
>
> Justin Lu has updated the pull request incrementally with two additional commits since the last revision:
> 
>  - Remove comment typo
>  - Remove test wrapper

src/java.base/share/classes/java/text/DigitList.java line 796:

> 794:         temp.append("x10^");
> 795:         temp.append(decimalAt);
> 796:         return temp.toString();

This could use chained method calls and avoid the local variable completely.

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

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


More information about the i18n-dev mailing list