RFR: 8330694: Rename 'HeapRegion' to 'G1HeapRegion' [v10]

Lei Zaakjyu duke at openjdk.org
Sun May 12 03:07:15 UTC 2024


On Sun, 12 May 2024 02:55:45 GMT, Lei Zaakjyu <duke at openjdk.org> wrote:

>> follow up 8267941
>
> Lei Zaakjyu has updated the pull request incrementally with one additional commit since the last revision:
> 
>   rename

Should we also rename 'HeapRegionType' to 'G1HeapRegionType', then rename the current 'G1HeapRegionType' to 'G1 HeapRegionTypeEnum'?

src/jdk.hotspot.agent/share/classes/sun/jvm/hotspot/gc/g1/G1CollectedHeap.java line 131:

> 129:             if (hr.isInRegion(addr)) {
> 130:                 return hr;
> 131:             }

Since these three methods are G1 specific, I'd prefer not to add the 'g1' prefix.

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

PR Review: https://git.openjdk.org/jdk/pull/18871#pullrequestreview-2051282649
PR Review Comment: https://git.openjdk.org/jdk/pull/18871#discussion_r1597540754


More information about the serviceability-dev mailing list