RFR(S) 8038404: Move object_iterate_mem from Space to CMS since it is only ever used by CMS

[Thomas Schatzl]

Hi,

On Thu, 2014-03-27 at 13:35 +0100, Mikael Gerdin wrote:
> On Thursday 27 March 2014 11.13.50 Stefan Karlsson wrote:
> > On 2014-03-27 08:55, Thomas Schatzl wrote:
> > > On Wed, 2014-03-26 at 17:21 +0100, Mikael Gerdin wrote:
> > >> I'm doing some cleanups in the "Space" classes to simplify the code for
> > >> the G1 class unloading work[1].
> > >> Here's a change to move CMS specific code to CompactibleFreeListSpace:
> > >> 
> > >> Bug: https://bugs.openjdk.java.net/browse/JDK-8038404
> > >> Webrev: http://cr.openjdk.java.net/~mgerdin/8038404/webrev.0
> > > 
> > > - Could you also move the documentation of Space::object_iterate_mem()
> > > to CompactibleFreeListSpace::object_iterate_mem()?
> > > 
> > > - the UpwardsObjectClosure also seems to be only used by
> > > Space::object_iterate_mem() too, and hence CMS. Is there some place to
> > > move it from the generic iterator.hpp to a CMS file?
> 
> I moved UpwardsObjectClosure to concurrentMarkSweepGeneration.hpp
> 
> New webrev: http://cr.openjdk.java.net/~mgerdin/8038404/webrev.1
> Incremental webrev: http://cr.openjdk.java.net/~mgerdin/8038404/webrev.0.to.1

Looks good.

Thank you,
Thomas