From: Eric S. Raymond <esr@thyrsus.com>
Date: Mon, 8 Oct 2012 15:58:36 +0000 (-0400)
Subject: Merge some minor cleanups by Peter Scott.
X-Git-Tag: 1.10~16
X-Git-Url: http://git.tremily.us/?a=commitdiff_plain;h=ea93bbc8c00a69c17ab51e01fffe6cf32c624047;p=irker.git

Merge some minor cleanups by Peter Scott.
---

diff --git a/irkerd b/irkerd
index 6ca6363..e565242 100755
--- a/irkerd
+++ b/irkerd
@@ -355,7 +355,7 @@ class Dispatcher:
         return len(self.connections) > 0
     def last_xmit(self):
         "Return the time of the most recent transmission."
-        return max([x.last_xmit for x in self.connections])
+        return max(x.last_xmit for x in self.connections)
 
 class Irker:
     "Persistent IRC multiplexer."
@@ -485,13 +485,8 @@ class Irker:
                             # choose the one longest idle on the
                             # assumption that message activity is likely
                             # to be clumpy.
-                            oldest = None
-                            oldtime = float("inf")
                             if len(self.servers) >= CONNECTION_MAX:
-                                for (name, server) in self.servers.items():
-                                    if server.last_xmit() < oldtime:
-                                        oldest = name
-                                        oldtime = server.last_xmit()
+                                oldest = min(self.servers.keys(), key=lambda name: self.servers[name].last_xmit())
                                 del self.servers[oldest]
         except ValueError:
             self.logerr("can't recognize JSON on input: %r" % line)