From 6500f68f1fd0614c7165fc9bea6d37a3cf17a8b4 Mon Sep 17 00:00:00 2001
From: Behdad Esfahbod <behdad@behdad.org>
Date: Fri, 6 Aug 2021 18:46:31 -0600
Subject: [PATCH] [util] Change hb-subset to use face_options_t instead of
 font_options_t

---
 util/hb-subset.cc      | 9 +++++----
 util/main-font-text.hh | 3 +--
 2 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/util/hb-subset.cc b/util/hb-subset.cc
index 7bd8ab1eb..4519c38b3 100644
--- a/util/hb-subset.cc
+++ b/util/hb-subset.cc
@@ -30,6 +30,7 @@
 #include "main-font-text.hh"
 #include "hb-subset.h"
 
+// XXX Remove eventually
 const unsigned DEFAULT_FONT_SIZE = FONT_SIZE_NONE;
 const unsigned SUBPIXEL_BITS = 0;
 
@@ -45,9 +46,9 @@ struct subset_consumer_t
     subset_options.add_options (parser);
   }
 
-  void init (const font_options_t *font_opts)
+  void init (const face_options_t *face_opts)
   {
-    face = hb_face_reference (hb_font_get_face (font_opts->get_font ()));
+    face = hb_face_reference (face_opts->get_face ());
     input = hb_subset_input_reference (subset_options.get_input ());
   }
 
@@ -98,7 +99,7 @@ struct subset_consumer_t
     return true;
   }
 
-  void finish (const font_options_t *font_opts)
+  void finish (const face_options_t *face_opts)
   {
     hb_face_t *new_face = nullptr;
     for (unsigned i = 0; i < subset_options.num_iterations; i++)
@@ -134,7 +135,7 @@ struct subset_consumer_t
 int
 main (int argc, char **argv)
 {
-  using driver_t = main_font_text_t<subset_consumer_t>;
+  using driver_t = main_font_text_t<subset_consumer_t, face_options_t>;
 
   if (argc == 2 && !strcmp (argv[1], "--batch"))
   {
diff --git a/util/main-font-text.hh b/util/main-font-text.hh
index 2bca9251c..f45f2c034 100644
--- a/util/main-font-text.hh
+++ b/util/main-font-text.hh
@@ -47,7 +47,7 @@ locale_to_utf8 (char *s)
   return t;
 }
 
-template <typename consumer_t>
+template <typename consumer_t, typename font_options_t=font_options_t, typename text_options_t=text_options_t>
 struct main_font_text_t
 {
   int
@@ -88,4 +88,3 @@ struct main_font_text_t
 };
 
 #endif
-