From 4a5c992b1a61f8d48b11e6310e4d036e5952f233 Mon Sep 17 00:00:00 2001 From: Dave Halter Date: Sat, 10 Aug 2019 14:41:47 +0200 Subject: [PATCH] Remove an unnecessary isinstance usage --- jedi/plugins/stdlib.py | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/jedi/plugins/stdlib.py b/jedi/plugins/stdlib.py index 62d3f3fa..2f2608fb 100644 --- a/jedi/plugins/stdlib.py +++ b/jedi/plugins/stdlib.py @@ -445,14 +445,16 @@ def collections_namedtuple(obj, arguments, callback): if not param_contexts: return NO_CONTEXTS _fields = list(param_contexts)[0] - if isinstance(_fields, compiled.CompiledValue): - fields = force_unicode(_fields.get_safe_value()).replace(',', ' ').split() + string = get_str_or_none(_fields) + if string is not None: + fields = force_unicode(string).replace(',', ' ').split() elif isinstance(_fields, iterable.Sequence): fields = [ - force_unicode(v.get_safe_value()) + force_unicode(get_str_or_none(v)) for lazy_context in _fields.py__iter__() - for v in lazy_context.infer() if is_string(v) + for v in lazy_context.infer() ] + fields = [f for f in fields if f is not None] else: return NO_CONTEXTS