You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mina.apache.org by el...@apache.org on 2012/02/09 11:40:09 UTC

svn commit: r1242266 - /mina/branches/2.0/mina-core/src/main/java/org/apache/mina/transport/socket/nio/NioProcessor.java

Author: elecharny
Date: Thu Feb  9 10:40:09 2012
New Revision: 1242266

URL: http://svn.apache.org/viewvc?rev=1242266&view=rev
Log:
Fixed DIRMINA-886 : the if() was broken, some '!' were missing

Modified:
    mina/branches/2.0/mina-core/src/main/java/org/apache/mina/transport/socket/nio/NioProcessor.java

Modified: mina/branches/2.0/mina-core/src/main/java/org/apache/mina/transport/socket/nio/NioProcessor.java
URL: http://svn.apache.org/viewvc/mina/branches/2.0/mina-core/src/main/java/org/apache/mina/transport/socket/nio/NioProcessor.java?rev=1242266&r1=1242265&r2=1242266&view=diff
==============================================================================
--- mina/branches/2.0/mina-core/src/main/java/org/apache/mina/transport/socket/nio/NioProcessor.java (original)
+++ mina/branches/2.0/mina-core/src/main/java/org/apache/mina/transport/socket/nio/NioProcessor.java Thu Feb  9 10:40:09 2012
@@ -164,9 +164,9 @@ public final class NioProcessor extends 
             for (SelectionKey key : keys) {
                 SelectableChannel channel = key.channel();
 
-                if ((((channel instanceof DatagramChannel) && ((DatagramChannel) channel)
+                if ((((channel instanceof DatagramChannel) && !((DatagramChannel) channel)
                         .isConnected()))
-                        || ((channel instanceof SocketChannel) && ((SocketChannel) channel)
+                        || ((channel instanceof SocketChannel) && !((SocketChannel) channel)
                                 .isConnected())) {
                     // The channel is not connected anymore. Cancel
                     // the associated key then.