On Thu, 2 May 2024 21:32:50 GMT, Chris Plummer <cjplum...@openjdk.org> wrote:
>> test/hotspot/jtreg/serviceability/sa/TestG1HeapRegion.java line 62: >> >>> 60: agent.attach(Integer.parseInt(pid)); >>> 61: G1CollectedHeap heap = >>> (G1CollectedHeap)VM.getVM().getUniverse().heap(); >>> 62: G1HeapRegion hr = heap.hrm().heapRegionIterator().next(); >> >> "g1HeapRegionIterator"? > > And here also it seems you agreed with this suggestion but the change was > never made. I think we can make these changes in later PRs in order to avoid making this one even larger. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/18871#discussion_r1588784617