[9] RFR (XS) JDK-8081771: ProcessTool.createJavaProcessBuilder() needs new addTestVmAndJavaOptions argument

serguei.spitsyn at oracle.com serguei.spitsyn at oracle.com
Thu Jun 4 00:01:10 UTC 2015


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



More information about the hotspot-runtime-dev mailing list