use pathops.simplify() and remember if original path direction

pathops.simplify() returns a copy so we don't need to make a copy ourselves.

'clockwise' option is defined in https://github.com/fonttools/skia-pathops/pull/31
This commit is contained in:
Cosimo Lupo 2020-09-27 17:31:30 +01:00
parent 0ceb146196
commit 015d8265d2
No known key found for this signature in database
GPG Key ID: 179A8F0895A02F4F

View File

@ -89,13 +89,10 @@ def removeOverlaps(
else: else:
path = skPathFromSimpleGlyph(glyphName, glyphSet) path = skPathFromSimpleGlyph(glyphName, glyphSet)
# duplicate path
path2 = pathops.Path(path)
# remove overlaps # remove overlaps
path2.simplify() path2 = pathops.simplify(path, clockwise=path.clockwise)
# replace TTGlyph if simplified copy is different # replace TTGlyph if simplified path is different
if path2 != path: if path2 != path:
glyfTable[glyphName] = glyph = ttfGlyphFromSkPath(path2) glyfTable[glyphName] = glyph = ttfGlyphFromSkPath(path2)
# also ensure hmtx LSB == glyph.xMin so glyph origin is at x=0 # also ensure hmtx LSB == glyph.xMin so glyph origin is at x=0