RFR (M): 8184346: Clean up G1CMBitmap

[Mikael Gerdin]

Hi Aleksey,

On 2017-07-20 20:50, Aleksey Shipilev wrote:
> On 07/20/2017 08:40 PM, Thomas Schatzl wrote:
>> Webrevs:
>> http://cr.openjdk.java.net/~tschatzl/8184346/webrev.2_to_3/ (diff)
>> http://cr.openjdk.java.net/~tschatzl/8184346/webrev.3/ (full)
> 

Looks fine to me too.

> Generally good, comments:
> 
>   *) Long log_debug, log_warning, assert lines in g1CollectedHeap.cpp,
> g1ConcurrentMark.cpp, g1ConcurrentMark.hpp
> 
>   *) It seems the field and method names are camel-cased and thus
> style-inconsistent with the rest of the code?
> 625   const G1CMBitMap* const prevMarkBitMap() const { return _prevMarkBitMap; }
> 626   G1CMBitMap* nextMarkBitMap() const { return _nextMarkBitMap; }

I think the idea is to perform that renaming in G1ConcurrentMark in a 
later change since this one tries to only concern G1CMBitMap.

/Mikael

> 
> Thanks,
> -Aleksey
>