<i18n dev> RFR: 8373566: Performance regression with java.text.MessageFormat subformat patterns [v4]

[Andrey Turbanov]

On Fri, 12 Dec 2025 23:28:08 GMT, Justin Lu <jlu at openjdk.org> wrote:

>> This PR addresses a performance degradation when constructing a new MessageFormat that contains a subformat with a pattern that is not a predefined style. This is due to internal logic which catches an IAE specifically for subformat patterns. Perf tests for choice and number are not grouped, since performance is directly tied to subformat creation.
>> 
>> M1 Mac - Perf (before):
>> 
>> 
>> MessageFormatterBench.testSubformatChoice  thrpt   15   865.430 ±  6.766  ops/ms
>> MessageFormatterBench.testSubformatNumber  thrpt   15  1021.808 ± 11.413  ops/ms
>> 
>> 
>> M1 Mac - Perf (after):
>> 
>> 
>> Benchmark                                   Mode  Cnt     Score    Error   Units
>> MessageFormatterBench.testSubformatChoice  thrpt   15  1671.033 ± 55.012  ops/ms
>> MessageFormatterBench.testSubformatNumber  thrpt   15  3275.727 ± 47.370  ops/ms
>
> Justin Lu has updated the pull request incrementally with one additional commit since the last revision:
> 
>   use SHORT.ordinal()

src/java.base/share/classes/java/text/MessageFormat.java line 2004:

> 2002:             var styles = values();
> 2003:             // Match starting at the pre-defined styles -> [SHORT:]
> 2004:             for (int i = SHORT.ordinal(); i < styles.length; i ++) {

Suggestion:

            for (int i = SHORT.ordinal(); i < styles.length; i++) {

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

PR Review Comment: https://git.openjdk.org/jdk/pull/28806#discussion_r2620785423