On 06/02/2020 13:48, Claes Redestad wrote:
:
New webrev:
http://cr.openjdk.java.net/~redestad/8237878/open.01/
The archiving looks good but I'd prefer if this patch didn't change the
usages of Function<String, ClassLoader> to ModuleLoaderMap.Mapper -
that's an implementation class that should not be used outside of the
ModuleLoaderMap (it should be private but there is special check in the
Module code that prevents this).
The ModuleLoaderMap constructor needs a comment so that future
maintainers know that it maps the modules in the given configuration to
the built-in class loaders. Also would be good if the declaration of map
were changed back to Map<String, Integer> and a // comment to make it
clear that it maps the module name to an index.
-Alan
