On Fri, 25 Jul 2025 12:08:38 GMT, Michael McMahon <[email protected]> wrote:
> Hi, > > This is a fix for 8362581 caused by the implementation for > https://bugs.openjdk.org/browse/JDK-8348986 > An incorrect check was added to the method Exceptions.ioException() which > wasn't noticed partly > because that file was renamed, and webrev doesn't show the diffs side by side. > The effect of this change is to restore the method to pretty much what it was > before, but taking account > of the changed categorisation of exceptions. > > A test is not included since this was caught by an existing regression test. > > Thanks, > Michael test/jdk/java/nio/channels/SocketChannel/OpenLeak.java line 25: > 23: > 24: /* @test > 25: * @bug 6548464 8362581 I think you can drop this change to the test, it only accidentally uncovered this issue. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/26478#discussion_r2231034209
