From faa2d015934b4a6a66e3db75b309b63c12ffe348 Mon Sep 17 00:00:00 2001 From: Dave Halter Date: Tue, 26 Sep 2017 18:36:10 +0200 Subject: [PATCH] The memoize decorator doesn't need to magically cache generators as lists. This makes no sense at all. Explicit is better than implicit. --- jedi/evaluate/cache.py | 4 ---- 1 file changed, 4 deletions(-) diff --git a/jedi/evaluate/cache.py b/jedi/evaluate/cache.py index 37bd0e8a..b7c7cd7e 100644 --- a/jedi/evaluate/cache.py +++ b/jedi/evaluate/cache.py @@ -4,8 +4,6 @@ - ``CachedMetaClass`` uses ``_memoize_default`` to do the same with classes. """ -import inspect - _NO_DEFAULT = object() @@ -40,8 +38,6 @@ def _memoize_default(default=_NO_DEFAULT, evaluator_is_first_arg=False, second_a if default is not _NO_DEFAULT: memo[key] = default rv = function(obj, *args, **kwargs) - if inspect.isgenerator(rv): - rv = list(rv) memo[key] = rv return rv return wrapper