On Sat, 16 Dec 2023 01:11:44 GMT, Chris Plummer <cjplum...@openjdk.org> wrote:
>> src/hotspot/share/runtime/globals.hpp line 557: >> >>> 555: "Limit the number of heap dumps triggered by " >>> \ >>> 556: "HeapDumpBeforeFullGC or HeapDumpAfterFullGC " >>> \ >>> 557: "(0 means no limit)" ) >>> \ >> >> nit: should be no space before the last paren. > > I was wondering if it is worth having HeapDumpBeforeFullGC and > HeapDumpAfterFullGC also mention FullGCHeapDumpLimit. Just something simple > like "Also see FullGCHeapDumpLimit." Not necessary, but if you think it would > be useful then please add. > nit: should be no space before the last paren. fixed. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/16976#discussion_r1428685031