RFR: 8154455: Fix compilation issue in WindowsAsynchronousSocketChannelImpl

Chris Hegarty chris.hegarty at oracle.com
Mon Apr 18 14:44:51 UTC 2016


Looks fine.

-Chris.

On 18/04/16 15:34, Claes Redestad wrote:
> Hi,
>
> the patch I pushed for JDK-8154436 earlier was somehow incomplete. The
> below patch should resolve things, and is currently working its way
> through JPRT
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8154455
>
> /Claes
>
> diff -r 2b4600ed4530
> src/java.base/windows/classes/sun/nio/ch/WindowsAsynchronousSocketChannelImpl.java
>
> ---
> a/src/java.base/windows/classes/sun/nio/ch/WindowsAsynchronousSocketChannelImpl.java
> Mon Apr 18 16:12:35 2016 +0200
> +++
> b/src/java.base/windows/classes/sun/nio/ch/WindowsAsynchronousSocketChannelImpl.java
> Mon Apr 18 16:16:16 2016 +0200
> @@ -649,11 +649,7 @@
>           }
>
>           // initiate I/O
> -        if (Iocp.supportsThreadAgnosticIo()) {
> -            readTask.run();
> -        } else {
> -            Invoker.invokeOnThreadInThreadPool(this, readTask);
> -        }
> +        readTask.run();
>           return result;
>       }
>
> @@ -906,13 +902,8 @@
>               result.setTimeoutTask(timeoutTask);
>           }
>
> -        // initiate I/O (can only be done from thread in thread pool)
>           // initiate I/O
> -        if (Iocp.supportsThreadAgnosticIo()) {
> -            writeTask.run();
> -        } else {
> -            Invoker.invokeOnThreadInThreadPool(this, writeTask);
> -        }
> +        writeTask.run();
>           return result;
>       }
>


More information about the nio-dev mailing list