RFR: 8331402: G1: Remove is_active() calls in G1HRPrinter logging

Albert Mingkun Yang ayang at openjdk.org
Thu May 2 15:01:53 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

Marked as reviewed by ayang (Reviewer).

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

PR Review: https://git.openjdk.org/jdk/pull/19061#pullrequestreview-2036010018


More information about the hotspot-gc-dev mailing list