<i18n dev> RFR: 8261254: Initialize charset mapping data lazily

Alan Bateman alanb at openjdk.java.net
Mon Feb 8 08:38:40 UTC 2021


On Sun, 7 Feb 2021 19:08:18 GMT, Claes Redestad <redestad at openjdk.org> wrote:

> This patch refactor JDK internal charsets to initialize charset mapping data lazily when needed via holder classes. This means both a startup improvement in some cases, and possible throughput improvements for all DoubleByte-based Charsets.
> 
> Testing: tier1-3

I wouldn't expect enumerating all charsets with Charset::availableCharsets to be too common but moving the data to holder class looks okay. The missing "final" in a few places was an oversight.  The replacement of the foreach and method ref in getServicesCatalog with imperative code is disappointment but okay here.

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

Marked as reviewed by alanb (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/2449


More information about the i18n-dev mailing list