On 9/10/2010 7:08 AM, Tomas Hurka wrote:
src/share/vm/services/heapDumper.hpp
Any particular reason for changing the one HeapDumper ctr from
public to private? Just curious...
According to the comment from Mandy, the second constructor
HeapDumper(bool gc_before_heap_dump, bool print_to_tty)
is not used. So I removed it and added new private constructor with oome flag.
Thanks for the explanation!
The new webrev with above changes is here:
<http://cr.openjdk.java.net/~thurka/6765718/webrev.02>
Thumbs up on the latest webrev.
Dan
