Thanks everybody for the reviews! The fix is pushed. // Katja
----- Original Message ----- From: mandy.ch...@oracle.com To: daniel.fu...@oracle.com, yekaterina.kantser...@oracle.com, serviceability-dev@openjdk.java.net, alan.bate...@oracle.com Sent: Wednesday, February 11, 2015 6:06:18 PM GMT +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, 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 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.net/browse/JDK-8072856 >> webrev: http://cr.openjdk.java.net/~ykantser/8072856/webrev.00/ >> >> Thanks, >> Katja >