RFR: 8148973: Rename g1/concurrentMark.{hpp,cpp,inline.hpp} to g1/g1ConcurrentMark.{hpp,cpp,inline.hpp}

Erik Helin erik.helin at oracle.com
Thu Feb 4 09:53:45 UTC 2016


On 2016-02-04, Thomas Schatzl wrote:
> Hi Erik,
> 
> On Thu, 2016-02-04 at 09:36 +0100, Erik Helin wrote:
> > Hi all,
> > 
> > this patch renames the files g1/concurrentMark.{hpp,cpp,inline.hpp}
> > to
> > g1/g1ConcurrentMark.{hpp,cpp,inline.hpp} (the prefix g1 is added).
> > The
> > classes in files have also been prefixed with G1 where it makes sense
> > (e.g. ConcurrentMark is now G1ConcurrentMark).
> > 
> > Webrev:
> > http://cr.openjdk.java.net/~ehelin/8148973/00/
> > 
> > Testing:
> > - JPRT
> > - Using grep to verify that the names of the old classes are no
> > longer
> >   used (not even in comments).
> 
>   some initial comments:
> 
> - the defines in the hpp files need to be fixed too.
> 
> - copyright updates for concurrentMarkThread*, g1CollectedHeap.*hpp,
> g1CollectorPolicy, g1EvacFailure*, whitebox.cpp

Thanks for reviewing! Incremental patch at:

http://cr.openjdk.java.net/~ehelin/8148973/00-01/

Thanks,
Erik

> Thanks,
>   Thomas


More information about the hotspot-gc-dev mailing list