From 85bda448b12c1892706f11309ece7d17bf7908de Mon Sep 17 00:00:00 2001 From: Dave Halter Date: Tue, 28 Nov 2017 08:43:56 +0100 Subject: [PATCH] Simplify one if statement --- jedi/evaluate/context/instance.py | 12 +++++------- 1 file changed, 5 insertions(+), 7 deletions(-) diff --git a/jedi/evaluate/context/instance.py b/jedi/evaluate/context/instance.py index 96bbf194..2d62a56f 100644 --- a/jedi/evaluate/context/instance.py +++ b/jedi/evaluate/context/instance.py @@ -104,13 +104,11 @@ class AbstractInstanceContext(Context): origin_scope=None, include_self_names=True): if include_self_names: for cls in self.class_context.py__mro__(): - if isinstance(cls, compiled.CompiledObject): - if cls.tree_node is not None: - # In this case we're talking about a fake object, it - # doesn't make sense for normal compiled objects to - # search for self variables. - yield SelfNameFilter(self.evaluator, self, cls, origin_scope) - else: + if not isinstance(cls, compiled.CompiledObject) \ + or cls.tree_node is not None: + # In this case we're excluding compiled objects that are + # not fake objects. It doesn't make sense for normal + # compiled objects to search for self variables. yield SelfNameFilter(self.evaluator, self, cls, origin_scope) for cls in self.class_context.py__mro__():