RFR: 8278276: G1: Refine naming of G1GCParPhaseTimesTracker::_must_record [v2]
Thomas Schatzl
tschatzl at openjdk.java.net
Tue Dec 7 13:28:19 UTC 2021
On Tue, 7 Dec 2021 03:40:37 GMT, Hamlin Li <mli at openjdk.org> wrote:
>> This is a trivial improvement to refine the naming of G1GCParPhaseTimesTracker::_must_record to _initial_record.
>
> Hamlin Li has updated the pull request incrementally with one additional commit since the last revision:
>
> use allow_multiple_record
Lgtm. Please fix the issue with the copyright date before pushing.
src/hotspot/share/gc/g1/g1GCParPhaseTimesTracker.hpp line 2:
> 1: /*
> 2: * Copyright (c) 2021, Oracle and/or its affiliates. All rights reserved.
Suggestion:
* Copyright (c) 2020, 2021, Oracle and/or its affiliates. All rights reserved.
-------------
Marked as reviewed by tschatzl (Reviewer).
PR: https://git.openjdk.java.net/jdk/pull/6714
More information about the hotspot-gc-dev
mailing list