andreacavalli/netty5
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Remove close(..) call which gave us troubles with

Browse Source 
ClosedChannelException. See #142 and #138
This commit is contained in:
Norman Maurer 2012-01-11 17:59:30 +01:00
parent e2109b236b
commit 02be2523e0
1 changed files with 8 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
src/main/java/org/jboss/netty/channel/socket/nio/NioClientSocketPipelineSink.java
View File

@ -370,7 +370,14 @@ class NioClientSocketPipelineSink extends AbstractChannelSink {
ConnectException cause = null; ConnectException cause = null;
for (SelectionKey k: keys) { for (SelectionKey k: keys) {
if (!k.isValid()) { 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; continue;
} }