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

deprecated BaseDefinition.line_nr in favor of 'line'. thx @dbrgn for the hint.

Browse Source
This commit is contained in:
David Halter
2012-12-27 18:12:18 +01:00
parent db4b09e05e
commit 22fcfe5e19
1 changed files with 13 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
jedi/api_classes.py
View File

@@ -91,6 +91,14 @@ class BaseDefinition(object):
@property
def line_nr(self):
""" Use line instead!
TODO remove, deprecated! it's just here for backwards compatibility
"""
raise DeprecationWarning("Use line instead.")
return self.line
@property
def line(self):
""" The line where the definition occurs (starting with 1). """
return self.start_pos[0]
@@ -204,8 +212,8 @@ class Completion(BaseDefinition):
else:
desc = '.'.join(unicode(p) for p in self.path)
line_nr = '' if self.in_builtin_module else '@%s' % self.line_nr
return '%s: %s%s' % (t, desc, line_nr)
line = '' if self.in_builtin_module else '@%s' % self.line
return '%s: %s%s' % (t, desc, line)
def follow_definition(self):
"""
@@ -282,7 +290,7 @@ class Definition(BaseDefinition):
"""
if self.module_path.endswith('.py') \
and not isinstance(self.definition, parsing.Module):
position = '@%s' % (self.line_nr)
position = '@%s' % (self.line)
else:
# is a builtin or module
position = ''