review request: 4244896: (process) Provide System.getPid(), System.killProcess(String pid)

Alan Bateman Alan.Bateman at oracle.com
Fri Jun 1 07:46:15 UTC 2012


On 31/05/2012 17:48, Rob McKenna wrote:
> That link should be:
>
> http://cr.openjdk.java.net/~robm/4244896/webrev.04/
>
>     -Rob
I'm happy the spec has come good too. One small suggestion is to tweak 
this line:

"{@code Process} objects returned by {@link ProcessBuilder#start} and 
{@link Runtime#exec} are of type that overrides this method and so 
invoking this method will forcibly terminatethe process. "

to

"Invoking this method on {@code Process} objects returned by {@link 
ProcessBuilder#start} and {@link Runtime#exec} will forcibly terminate 
the process".

Alsoin this sentence:

"... is called, however this method may be chained to {@code waitFor()} 
if needed."

may be clearer if you put "This method may be ..." as a separate sentence.

Otherwise I think it reads well.I hope to review the implementation and 
test soon.

-Alan




More information about the core-libs-dev mailing list