RFR: 8237878: Improve ModuleLoaderMap datastructures

Alan Bateman Alan.Bateman at oracle.com
Sun Feb 9 16:49:26 UTC 2020


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


More information about the core-libs-dev mailing list