Hi,
Could I please have a review of this small fix.
bug: https://bugs.openjdk.java.net/browse/JDK-8072856
webrev: http://cr.openjdk.java.net/~ykantser/8072856/webrev.00/
Thanks,
Katja
Looks good!
-JB-
On 11.2.2015 17:42, Yekaterina Kantserova wrote:
Hi,
Could I please have a review of this small fix.
bug: https://bugs.openjdk.java.net/browse/JDK-8072856
webrev: http://cr.openjdk.java.net/~ykantser/8072856/webrev.00/
Thanks,
Katja
Looks good to me too.
JEP 102 Process Updates will provide an easier way to get the
current process PID but we don't have that yet :-)
best regards,
-- daniel
On 11/02/15 17:42, Yekaterina Kantserova wrote:
Hi,
Could I please have a review of this small fix.
bug: https://bugs.openjdk.java.n
+1
This is temporary. When the Process API is available, we can convert to
use the new public API.
Mandy
On 2/11/2015 8:57 AM, Daniel Fuchs wrote:
Looks good to me too.
JEP 102 Process Updates will provide an easier way to get the
current process PID but we don't have that yet :-)
best re
+01:00 Amsterdam / Berlin /
Bern / Rome / Stockholm / Vienna
Subject: Re: 8072856: Eliminate ProcessTools.getProcessId dependency on
sun.management.VMManagement
+1
This is temporary. When the Process API is available, we can convert to
use the new public API.
Mandy
On 2/11/2015 8:57 AM