Actually, the channel is never null

Author: Stephane Landelle

src/main/java/com/ning/http/client/providers/netty/channel/ChannelManager.java

@@ -317,17 +317,15 @@ public void close() {
     public void closeChannel(Channel channel) {
 
         // The channel may have already been removed from the future if a timeout occurred, and this method may be called just after.
-        if (channel != null) {
-            LOGGER.debug("Closing Channel {} ", channel);
-            try {
-                removeAll(channel);
-                Channels.setDiscard(channel);
-                channel.close();
-            } catch (Throwable t) {
-                LOGGER.debug("Error closing a connection", t);
-            }
-            openChannels.remove(channel);
+        LOGGER.debug("Closing Channel {} ", channel);
+        try {
+            removeAll(channel);
+            Channels.setDiscard(channel);
+            channel.close();
+        } catch (Throwable t) {
+            LOGGER.debug("Error closing a connection", t);
         }
+        openChannels.remove(channel);
     }
 
     public void abortChannelPreemption(String poolKey) {