On Wed, 17 Jul 2024 17:36:29 GMT, Naoto Sato <na...@openjdk.org> wrote:

> Proposing a new overload method for `Process#waitFor()` which takes a 
> `Duration` for the timeout value. This will reduce the possibility for making 
> mistakes with the `TimeUnit` in the other overload method. A corresponding 
> CSR has also been drafted.

src/java.base/share/classes/java/lang/Process.java line 481:

> 479:      * this method returns immediately with the value {@code false}.
> 480:      *
> 481:      * <p>The default implementation of this method polls the {@code 
> exitValue}

There was discussion in the CSR on moving this to an `@implSpec`, are you going 
to do that change?

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/20220#discussion_r1681550925

Reply via email to