RFR: 8331402: G1: Remove is_active() calls in G1HRPrinter logging
Guoxiong Li
gli at openjdk.org
Thu May 2 23:25:51 UTC 2024
On Thu, 2 May 2024 13:03:09 GMT, Thomas Schatzl <tschatzl at openjdk.org> wrote:
> Hi all,
>
> the `is_active()` call for the various G1HRPrinter methods is unnecessary as the `log_trace()` call will do the exact same check again anyway.
>
> Testing: gha, local compilation
>
> Based on #19058.
>
> Hth,
> Thomas
Looks good.
-------------
Marked as reviewed by gli (Committer).
PR Review: https://git.openjdk.org/jdk/pull/19061#pullrequestreview-2037111592
More information about the hotspot-gc-dev
mailing list