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

return True from SinglePos.prune_post_subset to not change semantics

Browse Source 
the useless subtable might have been there to stop the next subtable from applying...
https://github.com/fonttools/fonttools/pull/2313#issuecomment-844497422
This commit is contained in:
Cosimo Lupo 2021-05-20 10:01:58 +01:00
parent defe0f997a
commit 926f721832
1 changed files with 4 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
Lib/fontTools/subset/__init__.py
View File

@ -555,8 +555,9 @@ def subset_glyphs(self, s):
@_add_method(otTables.SinglePos)
def prune_post_subset(self, font, options):
if not self.Value:
return False
if self.Value is None:
assert self.ValueFormat == 0
return True
# Shrink ValueFormat
if self.Format == 1:
@ -575,7 +576,7 @@ def prune_post_subset(self, font, options):
self.Value = self.Value[0] if self.ValueFormat != 0 else None
del self.ValueCount
return bool(self.ValueFormat)
return True
@_add_method(otTables.PairPos)
def subset_glyphs(self, s):