Make PyFT2Font a subclass of FT2Font

QuLogic · QuLogic · commit d725cf945607 · 2025-07-24T15:05:39.000-04:00
diff --git a/src/ft2font.cpp b/src/ft2font.cpp
@@ -207,15 +207,13 @@ FT2Font::get_path(std::vector<double> &vertices, std::vector<unsigned char> &cod
 }
 
 FT2Font::FT2Font(long hinting_factor_, std::vector<FT2Font *> &fallback_list,
-                 FT2Font::WarnFunc warn, bool warn_if_used)
-    : ft_glyph_warn(warn), warn_if_used(warn_if_used), image({1, 1}), face(nullptr),
+                 bool warn_if_used)
+    : warn_if_used(warn_if_used), image({1, 1}), face(nullptr), fallbacks(fallback_list),
       hinting_factor(hinting_factor_),
       // set default kerning factor to 0, i.e., no kerning manipulation
       kerning_factor(0)
 {
     clear();
-    // Set fallbacks
-    std::copy(fallback_list.begin(), fallback_list.end(), std::back_inserter(fallbacks));
 }
 
 FT2Font::~FT2Font()
@@ -544,21 +542,19 @@ FT_UInt FT2Font::get_char_index(FT_ULong charcode, bool fallback = false)
     return FT_Get_Char_Index(ft_object->get_face(), charcode);
 }
 
-void FT2Font::get_width_height(long *width, long *height)
+std::tuple<long, long> FT2Font::get_width_height()
 {
-    *width = advance;
-    *height = bbox.yMax - bbox.yMin;
+    return {advance, bbox.yMax - bbox.yMin};
 }
 
 long FT2Font::get_descent()
 {
     return -bbox.yMin;
 }
 
-void FT2Font::get_bitmap_offset(long *x, long *y)
+std::tuple<long, long> FT2Font::get_bitmap_offset()
 {
-    *x = bbox.xMin;
-    *y = 0;
+    return {bbox.xMin, 0};
 }
 
 void FT2Font::draw_glyphs_to_bitmap(bool antialiased)
@@ -607,8 +603,11 @@ void FT2Font::draw_glyph_to_bitmap(
     draw_bitmap(im, &bitmap->bitmap, x + bitmap->left, y);
 }
 
-void FT2Font::get_glyph_name(unsigned int glyph_number, std::string &buffer)
+std::string FT2Font::get_glyph_name(unsigned int glyph_number)
 {
+    std::string buffer;
+    buffer.resize(128);
+
     if (!FT_HAS_GLYPH_NAMES(face)) {
         /* Note that this generated name must match the name that
            is generated by ttconv in ttfont_CharStrings_getname. */
@@ -625,6 +624,8 @@ void FT2Font::get_glyph_name(unsigned int glyph_number, std::string &buffer)
             buffer.resize(len);
         }
     }
+
+    return buffer;
 }
 
 long FT2Font::get_name_index(char *name)
diff --git a/src/ft2font.h b/src/ft2font.h
@@ -12,6 +12,7 @@
 #include <set>
 #include <string>
 #include <string_view>
+#include <tuple>
 #include <unordered_map>
 #include <vector>
 
@@ -96,11 +97,9 @@ extern FT_Library _ft2Library;
 
 class FT2Font
 {
-    typedef void (*WarnFunc)(FT_ULong charcode, std::set<FT_String*> family_names);
-
   public:
     FT2Font(long hinting_factor, std::vector<FT2Font *> &fallback_list,
-            WarnFunc warn, bool warn_if_used);
+            bool warn_if_used);
     virtual ~FT2Font();
     void open(FT_Open_Args &open_args);
     void close();
@@ -124,14 +123,14 @@ class FT2Font
                                  std::set<FT_String*> &glyph_seen_fonts,
                                  bool override);
     void load_glyph(FT_UInt glyph_index, FT_Int32 flags);
-    void get_width_height(long *width, long *height);
-    void get_bitmap_offset(long *x, long *y);
+    std::tuple<long, long> get_width_height();
+    std::tuple<long, long> get_bitmap_offset();
     long get_descent();
     void draw_glyphs_to_bitmap(bool antialiased);
     void draw_glyph_to_bitmap(
         py::array_t<uint8_t, py::array::c_style> im,
         int x, int y, size_t glyphInd, bool antialiased);
-    void get_glyph_name(unsigned int glyph_number, std::string &buffer);
+    std::string get_glyph_name(unsigned int glyph_number);
     long get_name_index(char *name);
     FT_UInt get_char_index(FT_ULong charcode, bool fallback);
     void get_path(std::vector<double> &vertices, std::vector<unsigned char> &codes);
@@ -167,8 +166,9 @@ class FT2Font
         return FT_HAS_KERNING(face);
     }
 
+  protected:
+    virtual void ft_glyph_warn(FT_ULong charcode, std::set<FT_String*> family_names) = 0;
   private:
-    WarnFunc ft_glyph_warn;
     bool warn_if_used;
     py::array_t<uint8_t, py::array::c_style> image;
     FT_Face face;
diff --git a/src/ft2font_wrapper.cpp b/src/ft2font_wrapper.cpp

Original file line number	Diff line number	Diff line change
`@@ -207,15 +207,13 @@ FT2Font::get_path(std::vector<double> &vertices, std::vector<unsigned char> &cod`
`207`	`207`	`}`
`208`	`208`
`209`	`209`	`FT2Font::FT2Font(long hinting_factor_, std::vector<FT2Font *> &fallback_list,`
`210`		`- FT2Font::WarnFunc warn, bool warn_if_used)`
`211`		`- : ft_glyph_warn(warn), warn_if_used(warn_if_used), image({1, 1}), face(nullptr),`
	`210`	`+ bool warn_if_used)`
	`211`	`+ : warn_if_used(warn_if_used), image({1, 1}), face(nullptr), fallbacks(fallback_list),`
`212`	`212`	`hinting_factor(hinting_factor_),`
`213`	`213`	`// set default kerning factor to 0, i.e., no kerning manipulation`
`214`	`214`	`kerning_factor(0)`
`215`	`215`	`{`
`216`	`216`	`clear();`
`217`		`- // Set fallbacks`
`218`		`- std::copy(fallback_list.begin(), fallback_list.end(), std::back_inserter(fallbacks));`
`219`	`217`	`}`
`220`	`218`
`221`	`219`	`FT2Font::~FT2Font()`
`@@ -544,21 +542,19 @@ FT_UInt FT2Font::get_char_index(FT_ULong charcode, bool fallback = false)`
`544`	`542`	`return FT_Get_Char_Index(ft_object->get_face(), charcode);`
`545`	`543`	`}`
`546`	`544`
`547`		`-void FT2Font::get_width_height(long width, long height)`
	`545`	`+std::tuple<long, long> FT2Font::get_width_height()`
`548`	`546`	`{`
`549`		`- *width = advance;`
`550`		`- *height = bbox.yMax - bbox.yMin;`
	`547`	`+ return {advance, bbox.yMax - bbox.yMin};`
`551`	`548`	`}`
`552`	`549`
`553`	`550`	`long FT2Font::get_descent()`
`554`	`551`	`{`
`555`	`552`	`return -bbox.yMin;`
`556`	`553`	`}`
`557`	`554`
`558`		`-void FT2Font::get_bitmap_offset(long x, long y)`
	`555`	`+std::tuple<long, long> FT2Font::get_bitmap_offset()`
`559`	`556`	`{`
`560`		`- *x = bbox.xMin;`
`561`		`- *y = 0;`
	`557`	`+ return {bbox.xMin, 0};`
`562`	`558`	`}`
`563`	`559`
`564`	`560`	`void FT2Font::draw_glyphs_to_bitmap(bool antialiased)`
`@@ -607,8 +603,11 @@ void FT2Font::draw_glyph_to_bitmap(`
`607`	`603`	`draw_bitmap(im, &bitmap->bitmap, x + bitmap->left, y);`
`608`	`604`	`}`
`609`	`605`
`610`		`-void FT2Font::get_glyph_name(unsigned int glyph_number, std::string &buffer)`
	`606`	`+std::string FT2Font::get_glyph_name(unsigned int glyph_number)`
`611`	`607`	`{`
	`608`	`+ std::string buffer;`
	`609`	`+ buffer.resize(128);`
	`610`	`+`
`612`	`611`	`if (!FT_HAS_GLYPH_NAMES(face)) {`
`613`	`612`	`/* Note that this generated name must match the name that`
`614`	`613`	`is generated by ttconv in ttfont_CharStrings_getname. */`
`@@ -625,6 +624,8 @@ void FT2Font::get_glyph_name(unsigned int glyph_number, std::string &buffer)`
`625`	`624`	`buffer.resize(len);`
`626`	`625`	`}`
`627`	`626`	`}`
	`627`	`+`
	`628`	`+ return buffer;`
`628`	`629`	`}`
`629`	`630`
`630`	`631`	`long FT2Font::get_name_index(char *name)`