RFR(xs): 8199431: Split up class Metaspace into a static and a non-static part

[Andrew Dinn]

On 14/03/18 13:42, Thomas Stüfe wrote:
> Thank you all for the feedback!
> 
> New webrev to address Coleens points:
> 
> complete: http://cr.openjdk.java.net/~stuefe/webrevs/8199431-split-class-metaspace-into-two-parts/webrev.00/webrev/

I think that is meant to be webrev.01 :-)

The white space is gone but I am not clear that you have correctly
addresseed Coleen's second point.

In this new version Metaspace is no longer a friend of SpaceManager but
ClassLoaderMetaspace has been made a friend in its place. If the last
version worked without the friend declaration for ClassLoaderMetaspace,
then why does this new version need to make ClassLoaderMetaspace a friend?

regards,


Andrew Dinn
-----------
Senior Principal Software Engineer
Red Hat UK Ltd
Registered in England and Wales under Company Registration No. 03798903
Directors: Michael Cunningham, Michael ("Mike") O'Neill, Eric Shander