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