> This PR is a resubmission after PR #21593 was rolled back, and the unsafe 
> offset overflow issue has been fixed.
> 
> 1) Move getChars methods of StringLatin1 and StringUTF16 to DecimalDigits to 
> reduce duplication.
> 
> 2) HexDigits and OctalDigits also include getCharsLatin1 and getCharsUTF16
> 
> 3) Putting these two methods into DecimalDigits can avoid the need to expose 
> them in JavaLangAccess
> Eliminate duplicate code in BigDecimal
> 
> 4) This PR will improve the performance of Integer/Long.toString and 
> StringBuilder.append(int/long) scenarios. This is because Unsafe.putByte is 
> used to eliminate array bounds checks, and of course this elimination is 
> safe. In previous versions, in Integer/Long.toString and 
> StringBuilder.append(int/long) scenarios, -COMPACT_STRING performed better 
> than +COMPACT_STRING. This is because StringUTF16.getChars uses 
> StringUTF16.putChar, which is similar to Unsafe.putChar, and there is no 
> bounds check.

Shaojin Wen has updated the pull request incrementally with one additional 
commit since the last revision:

  fix comments

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/22023/files
  - new: https://git.openjdk.org/jdk/pull/22023/files/7eb89c91..960e279e

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=22023&range=12
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=22023&range=11-12

  Stats: 6 lines in 1 file changed: 0 ins; 0 del; 6 mod
  Patch: https://git.openjdk.org/jdk/pull/22023.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/22023/head:pull/22023

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

Reply via email to