On Thu, 26 Feb 2026 22:28:28 GMT, David Holmes <[email protected]> wrote:

>> `Process` implements `Closable` since 
>> https://bugs.openjdk.org/browse/JDK-8364361
>> 
>> Although on second thought, I agree it is worth considering whether or not 
>> we want to do this here, as it will make backporting this change less 
>> straight forward.
>
> TIL ... okay but TBH it looks weird to me to use it this way.

No, autoclose is good practice, please leave it. It does a waitpid if you don't 
do `waitFor` before, so it prevents Zombies.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/29923#discussion_r2871759642

Reply via email to