It looks good to me.
Reviewed all together.
Thanks,
Serguei
Thanks,
Serguei
On 6/2/15 8:20 PM, Chris Plummer wrote:
Please review the following:
Webrev: http://cr.openjdk.java.net/~cjplummer/8054386/webrev.02/
Bug: https://bugs.openjdk.java.net/browse/JDK-8081771
This review only concerns the changes to ProcessTool.java. The
CDSJDITests and filemapp.cpp changes will be committed under CR
JDK-8054386, but they rely on this change to ProcessTool.java, so both
CRs will be pushed together. See the following thread for the
JDK-8054386 review:
http://mail.openjdk.java.net/pipermail/hotspot-runtime-dev/2015-June/014923.html
thanks,
Chris