Re: RFR: 8294397: Replace StringBuffer with StringBuilder within java.text [v4]

2022-10-03 Thread Justin Lu
On Mon, 3 Oct 2022 20:42:55 GMT, Jonathan Gibbons wrote: >> 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: > >

Re: RFR: 8294397: Replace StringBuffer with StringBuilder within java.text [v4]

2022-10-03 Thread Jonathan Gibbons
On Fri, 30 Sep 2022 20:08:10 GMT, Justin Lu 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

Re: RFR: 8294397: Replace StringBuffer with StringBuilder within java.text [v4]

2022-10-03 Thread Brent Christian
On Fri, 30 Sep 2022 20:08:10 GMT, Justin Lu 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

Re: RFR: 8294397: Replace StringBuffer with StringBuilder within java.text [v4]

2022-10-01 Thread Lance Andersen
On Fri, 30 Sep 2022 20:08:10 GMT, Justin Lu 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

Re: RFR: 8294397: Replace StringBuffer with StringBuilder within java.text [v4]

2022-09-30 Thread Naoto Sato
On Fri, 30 Sep 2022 20:08:10 GMT, Justin Lu 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

Re: RFR: 8294397: Replace StringBuffer with StringBuilder within java.text [v4]

2022-09-30 Thread Justin Lu
> 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 comme