RFR (XS) 8208186: SetHeapSamplingInterval handles 1 explicitly

JC Beyler jcbeyler at google.com
Tue Aug 28 21:43:59 UTC 2018


Hi all,

Here is another extra small webrev for review where the code handles the
interval 1 differently but this was an artifact of the original
implementation. We could remove the test altogether (the rest of the code
makes the 0 "sample everything" work by side-effect) but for
readability/maintainance, it is best to explicitly test for and return 0.

Webrev: http://cr.openjdk.java.net/~jcbeyler/8208186/webrev.00/
Bug: https://bugs.openjdk.java.net/browse/JDK-8208186

Thanks,
Jc
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/serviceability-dev/attachments/20180828/78c5c681/attachment.html>


More information about the serviceability-dev mailing list