RFR: 8331402: G1: Remove is_active() calls in G1HRPrinter logging [v2]
Thomas Schatzl
tschatzl at openjdk.org
Fri May 3 12:11:10 UTC 2024
> 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
Thomas Schatzl has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase.
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/19061/files
- new: https://git.openjdk.org/jdk/pull/19061/files/25f24c1a..25f24c1a
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=19061&range=01
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=19061&range=00-01
Stats: 0 lines in 0 files changed: 0 ins; 0 del; 0 mod
Patch: https://git.openjdk.org/jdk/pull/19061.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/19061/head:pull/19061
PR: https://git.openjdk.org/jdk/pull/19061
More information about the hotspot-gc-dev
mailing list