From baa664bd11a9723be2304cef8c8c710e8b2ce563 Mon Sep 17 00:00:00 2001 From: Dave Hoover Date: Thu, 8 Nov 2007 18:49:28 +0000 Subject: [PATCH] channel is a SocketChannel which is a subclass of SelectableChannel, so this instanceof can't ever be false. (And just above, we've already confirmed that channel != null, so this check wasn't falling back to protecting us from that) git-svn-id: svn+ssh://src.earth.threerings.net/narya/trunk@4867 542714f4-19e9-0310-aa3c-eee0fc999fb1 --- .../presents/server/net/ConnectionManager.java | 12 ------------ 1 file changed, 12 deletions(-) diff --git a/src/java/com/threerings/presents/server/net/ConnectionManager.java b/src/java/com/threerings/presents/server/net/ConnectionManager.java index 89f38b0a4..4414b2071 100644 --- a/src/java/com/threerings/presents/server/net/ConnectionManager.java +++ b/src/java/com/threerings/presents/server/net/ConnectionManager.java @@ -656,18 +656,6 @@ public class ConnectionManager extends LoopingThread return; } - if (!(channel instanceof SelectableChannel)) { - try { - log.warning("Provided with un-selectable socket as result of accept(), can't " + - "cope [channel=" + channel + "]."); - } catch (Error err) { - log.warning("Un-selectable channel also couldn't be printed."); - } - // stick a fork in the socket - channel.socket().close(); - return; - } - // log.fine("Accepted connection " + channel + "."); // create a new authing connection object to manage the authentication of this client