mirror of
https://git.lyx.org/repos/lyx.git
synced 2024-11-29 05:01:49 +00:00
Just a bit of safety here.
This 0 default for end is wrong. You should be able to do: find_token(lines, token, 0, 0) and have that return -1. As it is, this is equivalent to: find_token(lines, token, 0, len(lines)) But I am afraid to change the default, in case something in lyx_1.2.py relies upon it somehow. git-svn-id: svn://svn.lyx.org/lyx/lyx-devel/trunk@36125 a592a061-630c-0410-9148-cb99ea01b6c8
This commit is contained in:
parent
f93c44791d
commit
6cc1d31e38
@ -128,7 +128,7 @@ def find_tokens(lines, tokens, start, end = 0, exact = False):
|
|||||||
the first element, in lines[start, end].
|
the first element, in lines[start, end].
|
||||||
|
|
||||||
Return -1 on failure."""
|
Return -1 on failure."""
|
||||||
if end == 0:
|
if end == 0 or end > len(lines):
|
||||||
end = len(lines)
|
end = len(lines)
|
||||||
|
|
||||||
for i in xrange(start, end):
|
for i in xrange(start, end):
|
||||||
@ -158,7 +158,7 @@ def find_re(lines, rexp, start, end = 0):
|
|||||||
|
|
||||||
Return -1 on failure."""
|
Return -1 on failure."""
|
||||||
|
|
||||||
if end == 0:
|
if end == 0 or end > len(lines):
|
||||||
end = len(lines)
|
end = len(lines)
|
||||||
for i in xrange(start, end):
|
for i in xrange(start, end):
|
||||||
if rexp.match(lines[i]):
|
if rexp.match(lines[i]):
|
||||||
|
Loading…
Reference in New Issue
Block a user