On Wed, 31 Jul 2024 19:10:41 GMT, Henry Lin <d...@openjdk.org> wrote:
> Adds a command line option `-redact` to `jcmd`, `redact` to `jmap` and
> `-XX:+HeapDumpRedacted` enabling redacted heap dumps. When enabled, the
> output binary heap dump has zeroes written out in place of the original
> primitive values in the object fields. There is a new jtreg test
> `heapDumpRedactedTest.java` that tests that the fields are properly redacted.
test/hotspot/jtreg/serviceability/dcmd/gc/HeapDumpRedactedTest.java line 100:
> 98:
> 99: //static field
> 100: JavaThing staticInt = testClass.getStaticField("staticInt");
Suggestion:
JavaThing staticInt = testClass.getStaticField("staticInt");
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/20409#discussion_r1699665859
