RFR: 8264423: G1: Rename full gc attribute table states [v2]

[Thomas Schatzl]

On Wed, 14 Apr 2021 12:16:02 GMT, Thomas Schatzl <tschatzl at openjdk.org> wrote:

>> src/hotspot/share/gc/g1/g1FullGCHeapRegionAttr.hpp line 71:
>> 
>>> 69:   }
>>> 70: 
>>> 71:   bool is_compacted_or_marked_through(uint idx) const {
>> 
>> Should this be rename to is_compacted_or_not_marked_through?
>> As the condition is indeed == Compacted || == NotMarkedThrough.
>
> Yes, that's true. It should actually be called `is_compacted_or_not_marked_through`.

See the other comment from me about naming variants provided by now.

-------------

PR: https://git.openjdk.java.net/jdk/pull/3486