shadowfacts/fonttools
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Apply suggestions from code review

Browse Source 
Co-authored-by: Cosimo Lupo <cosimo@anthrotype.com>
This commit is contained in:
Simon Cozens 2020-09-09 15:28:20 +01:00 committed by GitHub
parent bc9c7b6b14
commit a2e3e9dce2
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
Lib/fontTools/varLib/interpolatable.py
View File

@ -119,7 +119,7 @@ def test(glyphsets, glyphs=None, names=None):
allNodeTypes.append(nodeTypes)
allVectors.append(contourVectors)
for contour in contourPens:
nodeTypes.append(tuple([ instruction[0] for instruction in contour.value ]))
nodeTypes.append(tuple(instruction[0] for instruction in contour.value))
stats = StatisticsPen(glyphset=glyphset)
contour.replay(stats)
size = abs(stats.area) ** .5 * .5
@ -135,12 +135,12 @@ def test(glyphsets, glyphs=None, names=None):
#print(vector)
# Check each master against the next one in the list.
for i,(m0,m1) in enumerate(zip(allNodeTypes[:-1],allNodeTypes[1:])):
for i, (m0, m1) in enumerate(zip(allNodeTypes[:-1], allNodeTypes[1:])):
if len(m0) != len(m1):
print('%s: %s+%s: Glyphs not compatible (wrong number of paths %i+%i)!!!!!' % (glyph_name, names[i], names[i+1], len(m0), len(m1)))
if m0 == m1:
continue
for pathIx, (nodes1, nodes2) in enumerate(zip(m0,m1)):
for pathIx, (nodes1, nodes2) in enumerate(zip(m0, m1)):
if nodes1 == nodes2:
continue
print('%s: %s+%s: Glyphs not compatible at path %i!!!!!' % (glyph_name, names[i], names[i+1], pathIx))