VimPlug/jedi-fork
1
0
Fork 0
forked from VimPlug/jedi
Code Pull Requests Activity

Fix the fundamentally wrong cache.

Browse Source
This commit is contained in:
Dave Halter
2016-10-16 04:04:31 +02:00
parent 129c669bc0
commit 4ca3556c3b
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
jedi/cache.py
View File

@@ -80,13 +80,13 @@ def time_cache(time_add_setting):
def memoize_method(method): def memoize_method(method):
"""A normal memoize function.""" """A normal memoize function."""
def wrapper(self, *args, **kwargs): def wrapper(self, *args, **kwargs):
dct = self.__dict__.setdefault('_memoize_method_dct', {}) cache_dict = self.__dict__.setdefault('_memoize_method_dct', {})
dct = cache_dict.setdefault(method, {})
key = (args, frozenset(kwargs.items())) key = (args, frozenset(kwargs.items()))
try: try:
return dct[key] return dct[key]
except KeyError: except KeyError:
result = method(self, *args, **kwargs) result = method(self, *args, **kwargs)
# TODO THIS IS FUNDAMENTALLY WRONG, METHOD SHOULD BE INCLUDED.
dct[key] = result dct[key] = result
return result return result
return wrapper return wrapper