From 52d6fdff397f9ea5e1e4860d9f056bfe6525984b Mon Sep 17 00:00:00 2001 From: Nick Drozd Date: Fri, 8 Mar 2019 17:57:40 -0600 Subject: [PATCH] Cut cache_generator Is cache_generator doing anything? Cutting it doesn't break anything as far as I can tell, and it Pylint even seems to go a little faster without it. --- astroid/context.py | 11 ----------- astroid/node_classes.py | 7 +------ 2 files changed, 1 insertion(+), 17 deletions(-) diff --git a/astroid/context.py b/astroid/context.py index 70a9208863..2e1c8c352f 100644 --- a/astroid/context.py +++ b/astroid/context.py @@ -105,17 +105,6 @@ def clone(self): clone.extra_context = self.extra_context return clone - def cache_generator(self, key, generator): - """Cache result of generator into dictionary - - Used to cache inference results""" - results = [] - for result in generator: - results.append(result) - yield result - - self.inferred[key] = tuple(results) - @contextlib.contextmanager def restore_path(self): path = set(self.path) diff --git a/astroid/node_classes.py b/astroid/node_classes.py index 204d80233a..f3dadc20a6 100644 --- a/astroid/node_classes.py +++ b/astroid/node_classes.py @@ -352,12 +352,7 @@ def infer(self, context=None, **kwargs): if not context: return self._infer(context, **kwargs) - key = (self, context.lookupname, context.callcontext, context.boundnode) - if key in context.inferred: - return iter(context.inferred[key]) - - gen = context.cache_generator(key, self._infer(context, **kwargs)) - return util.limit_inference(gen, MANAGER.max_inferable_values) + return util.limit_inference(self._infer(context, **kwargs), MANAGER.max_inferable_values) def _repr_name(self): """Get a name for nice representation.