RFR: 8273218: G1: Rename g1EvacuationInfo to g1EvacInfo

Thomas Schatzl tschatzl at openjdk.java.net
Wed Sep 1 08:15:47 UTC 2021


On Wed, 1 Sep 2021 05:37:52 GMT, Hamlin Li <mli at openjdk.org> wrote:

> This is a minor cleanup to rename g1EvacuationInfo to g1EvacInfo, as other similar files are named as EvacXxx.

Lgtm.

Note that because you were so fast posting this, I do not know what others think about the name change in this direction, but I think it's okay. Please wait for further feedback.

src/hotspot/share/gc/g1/g1Allocator.cpp line 29:

> 27: #include "gc/g1/g1AllocRegion.inline.hpp"
> 28: #include "gc/g1/g1EvacStats.inline.hpp"
> 29: #include "gc/g1/g1EvacInfo.hpp"

Alphabetical order changed.

src/hotspot/share/gc/g1/g1CollectedHeap.hpp line 39:

> 37: #include "gc/g1/g1EvacFailure.hpp"
> 38: #include "gc/g1/g1EvacStats.hpp"
> 39: #include "gc/g1/g1EvacInfo.hpp"

Include order

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

Marked as reviewed by tschatzl (Reviewer).

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



More information about the hotspot-gc-dev mailing list