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

Naoto Sato naoto at openjdk.java.net
Mon Feb 8 17:42:42 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

Looks good to me. I initially thought of the same impression as Alan's, i.e., enumerating all charsets is not that common (in fact, same applies to Locale, and I once explored usages and it ended all up in test cases), but if real apps are using it, this is the right way to do so.


Marked as reviewed by naoto (Reviewer).

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

More information about the i18n-dev mailing list