<i18n dev> RFR: 8372460: Use EnumMap instead of HashMap for DateTimeFormatter parsing to improve performance [v7]

Shaojin Wen swen at openjdk.org
Fri Dec 19 02:43:01 UTC 2025


On Wed, 17 Dec 2025 09:01:09 GMT, Johannes Döbler <duke at openjdk.org> wrote:

>> Shaojin Wen has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   remove redundant checkField
>
> src/java.base/share/classes/java/time/format/Parsed.java line 175:
> 
>> 173:      */
>> 174:     @SuppressWarnings("unchecked")
>> 175:     Parsed(boolean onlyChronoField) {
> 
> If you know that only ChronoFields are used then imho the loop over the entries of `fieldValues` in method `resolveFields`  can be skipped (line 290ff).

Good suggestion, but that should be a separate PR

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

PR Review Comment: https://git.openjdk.org/jdk/pull/28471#discussion_r2633378092


More information about the i18n-dev mailing list