RFR: 8364361: [process] java.lang.Process should implement close and be AutoCloseable [v4]

Roger Riggs rriggs at openjdk.org
Wed Sep 3 16:19:45 UTC 2025


On Wed, 20 Aug 2025 07:54:46 GMT, Volkan Yazici <vyazici at openjdk.org> wrote:

>> Roger Riggs has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Revisited handling of IOExceptions to throw instead of completely ignoring or logging.
>
> test/jdk/java/lang/Process/ProcessCloseTest.java line 386:
> 
>> 384:                     break;
>> 385:                 } catch (InterruptedException ie) {
>> 386:                     // retry above
> 
> Are we sure about completely discarding the interrupt?

No interrupts are expected, the test is modified to consider InterruptedException as a test failure.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/26649#discussion_r2319474197


More information about the core-libs-dev mailing list