Check that Harfbuzz is at least version 4.1.0 to subset fonts

Fix #2185.
Kozea · Jun 17, 2024 · 8273f65 · 8273f65
1 parent e8540e7
commit 8273f65
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/weasyprint/pdf/fonts.py b/weasyprint/pdf/fonts.py
@@ -180,7 +180,8 @@ def subset(self, cmap, hinting):
         if not cmap:
             return
 
-        if harfbuzz_subset:
+        if harfbuzz_subset and harfbuzz.hb_version_atleast(4, 1, 0):
+            # 4.1.0 is required for hb_set_add_sorted_array.
             self._harfbuzz_subset(cmap, hinting)
         else:
             self._fonttools_subset(cmap, hinting)

diff --git a/weasyprint/text/ffi.py b/weasyprint/text/ffi.py
@@ -12,6 +12,7 @@
     typedef ... hb_font_t;
     typedef ... hb_face_t;
     typedef ... hb_blob_t;
+    typedef int hb_bool_t;
     typedef uint32_t hb_tag_t;
     typedef uint32_t hb_codepoint_t;
     hb_tag_t hb_tag_from_string (const char *str, int len);
@@ -30,7 +31,8 @@
         hb_tag_t *table_tags);
     unsigned int hb_face_get_glyph_count (const hb_face_t *face);
     hb_blob_t * hb_face_reference_table (const hb_face_t *face, hb_tag_t tag);
-
+    hb_bool_t hb_version_atleast (
+        unsigned int major, unsigned int minor, unsigned int micro);
 
     // HarfBuzz Subset