JDK-8331402 : G1: Remove is_active() calls in G1HRPrinter logging
  • Type: Enhancement
  • Component: hotspot
  • Sub-Component: gc
  • Affected Version: 23
  • Priority: P4
  • Status: Resolved
  • Resolution: Fixed
  • Submitted: 2024-04-30
  • Updated: 2024-05-09
  • Resolved: 2024-05-03
The Version table provides details related to the release that this issue/RFE will be addressed.

Unresolved : Release in which this issue/RFE will be addressed.
Resolved: Release in which this issue/RFE has been resolved.
Fixed : Release in which this issue/RFE has been fixed. The release containing this fix may be available for download as an Early Access Release or a General Availability Release.

To download the current JDK release, click here.
JDK 23
23 b22Fixed
Related Reports
Relates :  
JDK-8331401 - G1: Make G1HRPrinter AllStatic
Description
All G1HRPrinter methods use is_active() before printing the message.

That separate is_active() call duplicates the existing check if the log message is active in the log_trace() macro.

So the is_active() calls can be removed.
Comments
Changeset: 58ef9e48 Author: Thomas Schatzl <tschatzl@openjdk.org> Date: 2024-05-03 12:33:41 +0000 URL: https://git.openjdk.org/jdk/commit/58ef9e4805c0cc78935eb5a1c82ae10411d52e85
03-05-2024
A pull request was submitted for review. URL: https://git.openjdk.org/jdk/pull/19061 Date: 2024-05-02 13:03:09 +0000
02-05-2024