From 1cf4f3e0830eb45bd9d96fdfcdea15d2fb1af8f2 Mon Sep 17 00:00:00 2001
From: Behdad Esfahbod <behdad@behdad.org>
Date: Sat, 26 Nov 2022 18:15:28 -0700
Subject: [PATCH] [subset-cff] More comment

---
 src/hb-subset-cff-common.hh | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/src/hb-subset-cff-common.hh b/src/hb-subset-cff-common.hh
index d53dccfe1..95dc5f99b 100644
--- a/src/hb-subset-cff-common.hh
+++ b/src/hb-subset-cff-common.hh
@@ -353,7 +353,9 @@ struct parsed_cs_str_t : parsed_values_t<parsed_cs_op_t>
 	values[parsed_len-1].set_skip ();
 
 	/* Note(behdad): Alternatively, we can pop the subr number completely instead?
-	 * Seems to work. */
+	 * Seems to work. Then we can remove set_skip/is_skip completely. That would
+	 * change the design of this object that the ops won't be contiguous anymore.
+	 * I don't think that's a problem. Maybe switch this one day. */
         //values.pop ();
       }