RFR: 8159334: ModuleDescriptor retains overlapping sets for all and concealed packages

[Alan Bateman]

On 13/06/2016 08:37, Claes Redestad wrote:

> Hi,
>
> please review this patch to remove the set of concealed packages from 
> ModuleDescriptor and instead only calculate it on demand.
>
> Webrev: http://cr.openjdk.java.net/~redestad/8159334/webrev.01/
> Bug: https://bugs.openjdk.java.net/browse/JDK-8159334
>
> The conceals() method is currently used primarily by tools (jmod, 
> jlink), but never during startup or normal execution.  Removing 
> creation of it drops around 35Kb of heap data from a hello world 
> program (on a JDK image) as well as giving a small improvement to 
> startup time by virtue of creating fewer sets during bootstrap.
This looks okay and we can judge, in time, whether conceals needs to be 
cached. The patch has a couple of imports that aren't needed.

-Alan