From 02be2523e0da44f5e0432146419056ec9609b6da Mon Sep 17 00:00:00 2001
From: Norman Maurer <norman@apache.org>
Date: Wed, 11 Jan 2012 17:59:30 +0100
Subject: [PATCH] Remove close(..) call which gave us troubles with
 ClosedChannelException. See #142 and #138

---
 .../channel/socket/nio/NioClientSocketPipelineSink.java  | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

diff --git a/src/main/java/org/jboss/netty/channel/socket/nio/NioClientSocketPipelineSink.java b/src/main/java/org/jboss/netty/channel/socket/nio/NioClientSocketPipelineSink.java
index 55f904982c..01e0085615 100644
--- a/src/main/java/org/jboss/netty/channel/socket/nio/NioClientSocketPipelineSink.java
+++ b/src/main/java/org/jboss/netty/channel/socket/nio/NioClientSocketPipelineSink.java
@@ -370,7 +370,14 @@ class NioClientSocketPipelineSink extends AbstractChannelSink {
             ConnectException cause = null;
             for (SelectionKey k: keys) {
                 if (!k.isValid()) {
-                    close(k);
+                    // Comment the close call again as it gave us major problems
+                    // with ClosedChannelExceptions.
+                    //
+                    // See:
+                    // * https://github.com/netty/netty/issues/142
+                    // * https://github.com/netty/netty/issues/138
+                    //
+                    // close(k);
                     continue;
                 }