RFR: JDK-8327769: jcmd GC.heap_dump without options should write to location given by -XX:HeapDumpPath, if set [v6]

Chris Plummer cjplummer at openjdk.org
Sat Mar 16 22:18:46 UTC 2024


On Fri, 15 Mar 2024 09:01:20 GMT, Matthias Baesken <mbaesken at openjdk.org> wrote:

> Hi Chris, thanks for the comments-
> 
> > Also, if you are cleaning up this text, I would suggest changing "is on" to "is enabled". Same for HeapDumpGzipLevel below.
> 
> I changed the two locations. btw. seems we have more of those in globals.hpp . See the comments related to PrintNMTStatistics and LogFile (where the "is on" is used as well).

I didn't realize there were other occurrences. Up to you if you want to change some, all, or none.

-------------

PR Comment: https://git.openjdk.org/jdk/pull/18190#issuecomment-2000291906


More information about the hotspot-runtime-dev mailing list