RFR 9: 8003488 Add Process.getPid
Alan Bateman
Alan.Bateman at oracle.com
Thu May 22 15:01:21 UTC 2014
On 22/05/2014 14:49, roger riggs wrote:
> Hi,
>
> The webrev has been updated to more completely describe the pid:
> "The native process id is an identification number that the operating
> system assigns to the process."
>
> Any other comments?
>
> Webrev: http://cr.openjdk.java.net/~rriggs/webrev-getpid-8003488/
> <http://cr.openjdk.java.net/%7Erriggs/webrev-getpid-8003488/>
Roger and I discussed this one last week so I'm happy with the signature
and javadoc.
I see Ivan has picked up on the HANDLE <--> jlong and jlong_to_ptr is
normally the macro that we use for that.
A minor consistency point is that the getPid is added before the
constructor whereas the other overrides are below the constructor.
For the test then it would be good to add the issue number to the @bug line.
One question on the test as I'm not familiar with output of tasklist. If
2+ test were to run tasklist at the same time then could it confuse this
test? I'm just wondering if there is anything closer to echo $$.
Otherwise looks good to me.
-Alan.
More information about the core-libs-dev
mailing list