From a6c30e8c3579ef1e2e56915a612e78d3404fa7fc Mon Sep 17 00:00:00 2001
From: Behdad Esfahbod <behdad@behdad.org>
Date: Tue, 14 Nov 2023 11:08:14 -0700
Subject: [PATCH] [subset.cff] Change a type

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

The list type didn't make any sense. Use None instead.
---
 Lib/fontTools/subset/cff.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/Lib/fontTools/subset/cff.py b/Lib/fontTools/subset/cff.py
index dd79f6db3..03fc565b3 100644
--- a/Lib/fontTools/subset/cff.py
+++ b/Lib/fontTools/subset/cff.py
@@ -502,7 +502,7 @@ def remove_unused_subroutines(self):
         # Renumber glyph charstrings
         for g in font.charset:
             c, _ = cs.getItemAndSelector(g)
-            subrs = getattr(c.private, "Subrs", [])
+            subrs = getattr(c.private, "Subrs", None)
             c.subset_subroutines(subrs, font.GlobalSubrs)
 
         # Renumber subroutines themselves
@@ -511,7 +511,7 @@ def remove_unused_subroutines(self):
                 if not hasattr(font, "FDArray") and hasattr(font.Private, "Subrs"):
                     local_subrs = font.Private.Subrs
                 else:
-                    local_subrs = []
+                    local_subrs = None
             else:
                 local_subrs = subrs