On Wed, 20 Aug 2025 07:54:46 GMT, Volkan Yazici <[email protected]> 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
