On Sun, 7 Feb 2021 19:08:18 GMT, Claes Redestad <redes...@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.

src/java.base/share/classes/java/lang/ModuleLayer.java line 936:

> 934:                 for (Module m : nameToModule.values()) {
> 935:                     servicesCatalog.register(m);
> 936:                 }

Seems to be unrelated, but it's not a bad change.

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

Marked as reviewed by jkuhn (Author).

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

Reply via email to