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

line/column ValueError are now being raised if they are not valid in Script

Browse Source
This commit is contained in:
David Halter
2013-08-31 11:51:52 +04:30
parent 23ece5e4c8
commit 0d221ffebc
2 changed files with 8 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
jedi/api.py
View File

@@ -65,7 +65,13 @@ class Script(object):
lines.append('') lines.append('')
self._line = max(len(lines), 1) if line is None else line self._line = max(len(lines), 1) if line is None else line
self._column = len(lines[-1]) if column is None else column if not (0 < self._line <= len(lines)):
raise ValueError('`line` parameter is not in a valid range.')
line_len = len(lines[self._line - 1])
self._column = line_len if column is None else column
if not (0 <= self._column <= line_len):
raise ValueError('`column` parameter is not in a valid range.')
api_classes._clear_caches() api_classes._clear_caches()
debug.reset_time() debug.reset_time()

2
test/test_api.py
View File

@@ -50,4 +50,4 @@ def test_line_number_errors():
# ok # ok
api.Script(s, 1, 0) api.Script(s, 1, 0)
api.Script(s, 1, len(s) + 1) api.Script(s, 1, len(s))