<i18n dev> RFR: 8333396: Performance regression of DecimalFormat.format [v8]

lingjun-cg duke at openjdk.org
Wed Jun 19 02:17:11 UTC 2024


On Tue, 18 Jun 2024 20:39:30 GMT, Justin Lu <jlu at openjdk.org> wrote:

>> lingjun-cg has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   8333396: Performance regression of DecimalFormat.format
>
> src/java.base/share/classes/java/text/ChoiceFormat.java line 561:
> 
>> 559:             toAppendTo.append(choiceFormats[i]);
>> 560:         } catch (IOException ioe) {
>> 561:             throw new UncheckedIOException(ioe.getMessage(), ioe);
> 
> Perhaps `AssertionError` instead of `UncheckedIOException` is better suited here and in the other ocurrences.

This can be removed if using `StringBuf`.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/19513#discussion_r1645304136


More information about the i18n-dev mailing list