RFR: 8316150: Refactor get chars and string size [v7]

[Jim Laskey]

On Fri, 22 Sep 2023 11:37:18 GMT, Chen Liang <liach at openjdk.org> wrote:

>> 温绍锦 has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   restore HexDigits & OctalDigits
>
> src/java.base/share/classes/java/util/FormatItem.java line 148:
> 
>> 146:             int length = DecimalDigits.stringSize(value);
>> 147:             this.digits = new byte[length];
>> 148:             DecimalDigits.getCharsLatin1(value, length, this.digits);
> 
> Sorry missed this one in last round of review. This is wrong if you mix number fornat items and chinese segments in a String Template.

Try again (been on vacation). Not sure I agree with this refactoring. You need to account for UTF16, hence the PUT_CHAR_DIGIT. Recommend you revert and add getCharsLatin1 using the original implementation.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/15699#discussion_r1334295471