Commit b5bd09d9 authored by Yoshinori Okuji's avatar Yoshinori Okuji

If cache_duration is zero, set self.expires_at to None to avoid a needless comparison.

git-svn-id: https://svn.erp5.org/repos/public/erp5/trunk@29692 20353a03-c40f-0410-a6d1-a30d3c3de9de
parent 9bc0ed81
...@@ -49,7 +49,7 @@ class CacheEntry(object): ...@@ -49,7 +49,7 @@ class CacheEntry(object):
def __init__(self, value, cache_duration=None, calculation_time=0): def __init__(self, value, cache_duration=None, calculation_time=0):
self.value = value self.value = value
if cache_duration in (None, 0): if cache_duration in (None, 0):
self.expires_at = cache_duration self.expires_at = None
else: else:
self.expires_at = time.time() + cache_duration self.expires_at = time.time() + cache_duration
self._cache_hit_count = 0 self._cache_hit_count = 0
...@@ -60,7 +60,7 @@ class CacheEntry(object): ...@@ -60,7 +60,7 @@ class CacheEntry(object):
- None means allways expire - None means allways expire
- 0 means never expire - 0 means never expire
""" """
return self.expires_at is None or self.expires_at != 0 and self.expires_at < time.time() return self.expires_at is None or self.expires_at < time.time()
def markCacheHit(self, delta=1): def markCacheHit(self, delta=1):
""" mark a read to this cache entry """ """ mark a read to this cache entry """
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment