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

Thomas Schatzl thomas.schatzl at oracle.com
Thu Feb 4 09:09:33 UTC 2016


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,
  Thomas


More information about the hotspot-gc-dev mailing list