Skip to content

Refactored parts of the new ZGC parsing API to improve consistency#455

Merged
karianna merged 1 commit intomicrosoft:mainfrom
fthevenet:main
Oct 8, 2025
Merged

Refactored parts of the new ZGC parsing API to improve consistency#455
karianna merged 1 commit intomicrosoft:mainfrom
fthevenet:main

Conversation

@fthevenet
Copy link
Copy Markdown
Contributor

@fthevenet fthevenet commented Oct 7, 2025

Hi,

I would like to propose a few changes to the way some classes and class members are named in the ZGC parsing API.
Since #447 introduced a breaking change in this part of the API and has not yet be released, I think this is a good time to review a few inconsistencies that make discovery of the API more difficult.

Changes can be grouped as such:

  1. Changed incorrect references to "ReclaimStart/End" instead of "RelocateStart/End" in ZGCReclaimSummary
  2. Renamed occurrences of the term "Reclaim" to "Reclaimed" in many places, for consistency.
  3. Renamed the class OccupancySummary to ZGCUsedSummary for consistency
  4. Renamed methods get/setGenerationUsedSummary to get/setUsedSummary for consistency.

Thanks.

@fthevenet
Copy link
Copy Markdown
Contributor Author

@microsoft-github-policy-service agree company="IBM"

@karianna karianna merged commit 2b918d0 into microsoft:main Oct 8, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants