Small behaviour changes in select close

channels/base.py

@@ -130,6 +130,9 @@ class HandlingLayer(object):
         """Handling layer is forcing the channel to be unregistered, as because
         it has become closed or failed. This is called on already closed channel.
 
+        By the time this is called channel has been already removed from the
+        select layer.
+
         @param channel: channel that has failed
         @type channel: L{Channel}
         """

channels/sockets/sockets.py

@@ -303,9 +303,9 @@ class SelectHandlingLayer(HandlingLayer):
         """
         Channel unregister + channel close
         """
+        self.unregister_channel(channel)
         channel.on_closed() # this should close the channel
         self.on_closed(channel)
-        self.unregister_channel(channel)
 
     def select(self, timeout=5):
         self.on_iteration()