dragonwell-project / dragonwell21

GNU General Public License v2.0
61 stars 18 forks source link

[Backport] 8333396: Use StringBuilder internally for java.text.Format.* formatting #85

Closed lingjun-cg closed 1 month ago

lingjun-cg commented 1 month ago

[Backport] 8333396: Use StringBuilder internally for java.text.Format.* formatting

Summary: java.text.Format. use StringBuffer internally, but StringBuffer's bad performance character become obvious after obsoleting biased locking. So use StringBuilder if possible in java.text.Formating..

Testing: All java.text.format jtreg.

Reviewers: D-D-H, Yude Lin

Issue: https://github.com/dragonwell-project/dragonwell21/issues/83

linade commented 1 month ago

Is this a clean backport?

linade commented 1 month ago

Also does it change any public api that might break any existing dw21 applications

linade commented 1 month ago

lgtm, effectively clean backport without touching public api, only some description changes

lingjun-cg commented 1 month ago

Also does it change any public api that might break any existing dw21 applications

No public api changed.

lingjun-cg commented 1 month ago

Is this a clean backport?

Yes. There are only some minor conflicts at the topmost of file that contains "Copyright ".