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

Return None instead of raising an IndexError in get_next_leaf.

Browse Source
This commit is contained in:
Dave Halter
2017-02-03 17:26:02 +01:00
parent c5071f9f49
commit 647aec11a6
1 changed files with 1 additions and 13 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
jedi/parser/tree.py
View File

@@ -255,7 +255,7 @@ class Base(object):
if i == len(c) - 1: if i == len(c) - 1:
node = node.parent node = node.parent
if node.parent is None: if node.parent is None:
raise IndexError('Cannot access the next element of the last one.') return None
else: else:
node = c[i + 1] node = c[i + 1]
break break
@@ -566,18 +566,6 @@ class BaseNode(Base):
except AttributeError: except AttributeError:
return self.children[0] return self.children[0]
def get_next_leaf(self):
"""
Raises an IndexError if it's the last node. (Would be the module)
"""
c = self.parent.children
index = c.index(self)
if index == len(c) - 1:
# TODO WTF? recursion?
return self.get_next_leaf()
else:
return c[index + 1]
def last_leaf(self): def last_leaf(self):
return self.children[-1].last_leaf() return self.children[-1].last_leaf()