Skip to content

Commit

Permalink
[checkoutlinesufo] fix nested loop variable
Browse files Browse the repository at this point in the history
This is a fix for the following issue identified by LGTM:
Nested for statement uses loop variable 'point' of enclosing for statement.
  • Loading branch information
cjchapman committed Oct 2, 2020
1 parent 2c36ad1 commit 1cf389c
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions python/afdko/checkoutlinesufo.py
Original file line number Diff line number Diff line change
Expand Up @@ -927,10 +927,10 @@ def restore_contour_order(fixed_glyph, original_contours):
for j in old_index_list:
ci2, old_contour = old_list[j]
matched = False
for point in old_contour:
if point.segmentType is None:
for old_point in old_contour:
if old_point.segmentType is None:
continue
if (max_p.x == point.x) and (max_p.y == point.y):
if (max_p.x == old_point.x) and (max_p.y == old_point.y):
new_list[i] = None
order_list.append([ci2, ci])
matched = True
Expand Down

0 comments on commit 1cf389c

Please sign in to comment.