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