diff options
| author | Tres Seaver <tseaver@palladion.com> | 2009-04-29 17:40:44 +0000 |
|---|---|---|
| committer | Tres Seaver <tseaver@palladion.com> | 2009-04-29 17:40:44 +0000 |
| commit | d36067078e3f8ee91545193f765225edb89065fb (patch) | |
| tree | a19d2fc0f131aebc802943d119657697200f9b5a | |
| parent | 89aa8e7071a11a43b844cbda7c1a3314302731e9 (diff) | |
| download | pyramid-d36067078e3f8ee91545193f765225edb89065fb.tar.gz pyramid-d36067078e3f8ee91545193f765225edb89065fb.tar.bz2 pyramid-d36067078e3f8ee91545193f765225edb89065fb.zip | |
Remove concurrency wart.
| -rw-r--r-- | repoze/bfg/lru.py | 13 |
1 files changed, 4 insertions, 9 deletions
diff --git a/repoze/bfg/lru.py b/repoze/bfg/lru.py index b757207c8..947f8193f 100644 --- a/repoze/bfg/lru.py +++ b/repoze/bfg/lru.py @@ -4,9 +4,8 @@ import threading try: from functools import wraps -except ImportError: - # < 2.5 - from repoze.bfg.functional import wraps +except ImportError: # < 2.5 #pragma NO COVERAGE + from repoze.bfg.functional import wraps #pragma NO COVERAGE _marker = object() @@ -60,12 +59,8 @@ class LRUCache(object): lock.acquire() try: oldkey = current['key'] - if oldkey is not _marker: - try: - del data[oldkey] - except KeyError: - # XXX already deleted; seen in wild 5/16/2009? - pass + if oldkey in data: + del data[oldkey] current['key'] = key current['ref'] = True data[key] = (hand, val) |
