Cosmetics

celery/utils/functional.py

@@ -55,19 +55,13 @@ class LRUCache(UserDict):
 
     def update(self, *args, **kwargs):
         with self.mutex:
-            self.data.update(*args, **kwargs)
-
-            if not self.limit:
-                return
-
-            # pop additional items if dict growed too much
-            i = iter(self.data)  # start from bottom (LRU)
-            overflow = len(self.data) - self.limit
-
-            # negative overflow will lead to an empty list
-            to_pop = [next(i) for _ in xrange(overflow)]
-            for item in to_pop:
-                self.data.pop(item)
+            data, limit = self.data, self.limit
+            data.update(*args, **kwargs)
+            if limit and len(data) > limit:
+                # pop additional items in case limit exceeded
+                # negative overflow will lead to an empty list
+                for item in islice(iter(data), len(data) - limit):
+                    data.pop(item)
 
     def __setitem__(self, key, value):
         # remove least recently used key.