Actually, the channel is never null

Author: Stephane Landelle

providers/netty/src/main/java/org/asynchttpclient/providers/netty/channel/ChannelManager.java

@@ -308,19 +308,15 @@ public void close() {
 
     public void closeChannel(Channel channel) {
 
-        // The channel may have already been removed 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) {