On Fri, 1 May 2026 12:35:55 GMT, Viktor Klang <[email protected]> wrote:

>> src/java.base/share/classes/sun/util/locale/provider/LocaleResources.java 
>> line 700:
>> 
>>> 698:                 regionToSkeletonMap.getOrDefault(locale.getCountry(),
>>> 699:                     
>>> regionToSkeletonMap.getOrDefault(locale.getLanguage() + "-001",
>>> 700:                         regionToSkeletonMap.getOrDefault("001", 
>>> "h")))) :
>> 
>> Original code and the current code are performing 4 lookups even when one of 
>> the earlier lookup would produce a valid return value. Feels a bit painful :(
>
> Seconded

Very good point.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/31006#discussion_r3175033958

Reply via email to