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

[subset.cff] Change a type

Browse Source 
Doesn't fix, but related to
https://github.com/fonttools/fonttools/discussions/3332

The list type didn't make any sense. Use None instead.
This commit is contained in:
Behdad Esfahbod 2023-11-14 11:08:14 -07:00
parent 441228e85a
commit a6c30e8c35
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
Lib/fontTools/subset/cff.py
View File

@ -502,7 +502,7 @@ def remove_unused_subroutines(self):
# Renumber glyph charstrings # Renumber glyph charstrings
for g in font.charset: for g in font.charset:
c, _ = cs.getItemAndSelector(g) c, _ = cs.getItemAndSelector(g)
subrs = getattr(c.private, "Subrs", []) subrs = getattr(c.private, "Subrs", None)
c.subset_subroutines(subrs, font.GlobalSubrs) c.subset_subroutines(subrs, font.GlobalSubrs)
# Renumber subroutines themselves # Renumber subroutines themselves
@ -511,7 +511,7 @@ def remove_unused_subroutines(self):
if not hasattr(font, "FDArray") and hasattr(font.Private, "Subrs"): if not hasattr(font, "FDArray") and hasattr(font.Private, "Subrs"):
local_subrs = font.Private.Subrs local_subrs = font.Private.Subrs
else: else:
local_subrs = [] local_subrs = None
else: else:
local_subrs = subrs local_subrs = subrs