RFR: 8237878: Improve ModuleLoaderMap datastructures
Alan.Bateman at oracle.com
Mon Feb 10 11:34:15 UTC 2020
On 10/02/2020 09:04, Claes Redestad wrote:
> So how about:
Thanks for restoring the use of Function<String, ClassLoader>. Changing
Module::defineClass to invoke a method on ModuleLoaderMap is okay but
the method needs to something like "isBuiltinMapper" because it tests if
the function is a built-in mapper used for the boot layer (or child
layers created when we need to dynamically augment the set of platform
Minor nit but I think the comment on the Mapper constructor would say
that it creates a Mapper to map the modules in the given Configuration
to the built-in class loaders.
The rest looks good to me.
More information about the core-libs-dev