Hi Chris,

Looks good!


Yasumasa


On 2020/04/03 5:46, Chris Plummer wrote:
Hello,

Please review the following:

https://bugs.openjdk.java.net/browse/JDK-8240989
http://cr.openjdk.java.net/~cjplummer/8240989/webrev.01/

See the CR for details.

For the test I used ClhsdbField.java as a simple clhsdb template. To verify the 
dumped heap I added the call to printStackTraces(), which came from 
test/jdk/sun/tools/jhsdb/HeapDumpTest.java.

thanks,

Chris

Reply via email to