Looks fine to me. It is unusual to have two noreg labels on a bug. Is that acceptable? I would probably choose one or the other. noreg-hard seems more appropriate to me.

--Sean

On 10/15/18 9:34 PM, Weijun Wang wrote:
Please take a review at

    http://cr.openjdk.java.net/~weijun/8212216/webrev.00/

This bug is reported and fixed by Nico Williams <n...@twosigma.com>.

No regression test, memory leak.

Thanks
Max

Reply via email to