Looks OK.
   164         // sanity check to have non-zero usage
should be changed to ?
   164         // sanity check to have non-negative usage

Shanliang

Jaroslav Bachorik wrote:
Please, review this simple test fix.

Issue:  https://bugs.openjdk.java.net/browse/JDK-7112404
Webrev: http://cr.openjdk.java.net/~jbachorik/7112404/webrev.00

The tests assume the MemoryUsage#commited values to be positive (>0) while according to the MemoryUsage only negative values are invalid. Therefore the tests should check and fail only when MemoryUsage#commited is < 0.

Thanks,

-JB-

Reply via email to