8072856: Eliminate ProcessTools.getProcessId dependency on sun.management.VMManagement

2015-02-11 Thread Yekaterina Kantserova
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

Re: 8072856: Eliminate ProcessTools.getProcessId dependency on sun.management.VMManagement

2015-02-11 Thread Jaroslav Bachorik
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

Re: 8072856: Eliminate ProcessTools.getProcessId dependency on sun.management.VMManagement

2015-02-11 Thread Daniel Fuchs
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

Re: 8072856: Eliminate ProcessTools.getProcessId dependency on sun.management.VMManagement

2015-02-11 Thread Mandy Chung
+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

Re: 8072856: Eliminate ProcessTools.getProcessId dependency on sun.management.VMManagement

2015-02-12 Thread Yekaterina Kantserova
+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