Re: RFR(M): 8220628: Move the HeapMonitor library to C++

Wed, 13 Mar 2019 17:01:30 -0700

Hi Jc,

I do not see any issues, it looks good.
This is nice as the code became simpler.

Thanks,
Serguei


On 3/13/19 16:07, Jean Christophe Beyler wrote:
Hi all,

Could I get a review of:

I've not tried to do anything special here to keep the review simple; in the next webrev or two, I'll move more code to more C++ style and then work on diagnostic print-outs (in C++ :-)) to figure out the issues with the bugs related to these tests.

This passed testing on my dev machine and a submit repo run (which I'm not sure runs these test but still good to check).

Thanks!
Jc

Reply via email to