On Fri, 22 Jan 2021 06:51:59 GMT, Yasumasa Suenaga <[email protected]> wrote:
>> G1 heap summary (G1 Heap, summaries for each spaces) is shown on console
>> even though I chosen "Heap Parameters" menu on HSDB. It should be shown on
>> "Heap Parameters" window on HSDB.
>
> Yasumasa Suenaga 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. The pull request contains three additional
> commits since the last revision:
>
> - Update copyright year
> - Merge remote-tracking branch 'upstream/master' into JDK-8259009
> - G1 heap summary should be shown in "Heap Parameters" window on HSDB
I would prefer to make the order of parameters uniform in
`printG1HeapSummary(G1CollectedHeap g1h, PrintStream tty) {` too. I.e. there
does not seem to be a reason to have only this function have `tty` as second
parameter.
-------------
Changes requested by tschatzl (Reviewer).
PR: https://git.openjdk.java.net/jdk/pull/1911
