ft2font: Split layouting from set_text

The former may be used even on PS/PDF backend where nothing is rendered.

Author: QuLogic

src/ft2font.cpp

@@ -314,27 +314,12 @@ void FT2Font::set_kerning_factor(int factor)
     }
 }
 
-void FT2Font::set_text(
-    std::u32string_view text, double angle, FT_Int32 flags, std::vector<double> &xys)
+void FT2Font::layout(std::u32string_view text, FT_Int32 flags,
+                     std::set<FT_String*>& glyph_seen_fonts,
+                     std::vector<raqm_glyph_t>& glyphs)
 {
-    FT_Matrix matrix; /* transformation matrix */
-
-    angle = angle * (2 * M_PI / 360.0);
-
-    // this computes width and height in subpixels so we have to multiply by 64
-    double cosangle = cos(angle) * 0x10000L;
-    double sinangle = sin(angle) * 0x10000L;
-
-    matrix.xx = (FT_Fixed)cosangle;
-    matrix.xy = (FT_Fixed)-sinangle;
-    matrix.yx = (FT_Fixed)sinangle;
-    matrix.yy = (FT_Fixed)cosangle;
-
     clear();
 
-    bbox.xMin = bbox.yMin = 32000;
-    bbox.xMax = bbox.yMax = -32000;
-
     auto rq = raqm_create();
     if (!rq) {
         throw std::runtime_error("failed to compute text layout");
@@ -362,7 +347,6 @@ void FT2Font::set_text(
     }
 
     std::vector<std::pair<size_t, const FT_Face&>> face_substitutions;
-    std::set<FT_String*> glyph_seen_fonts;
     glyph_seen_fonts.insert(face->family_name);
 
     // Attempt to use fallback fonts if necessary.
@@ -416,9 +400,34 @@ void FT2Font::set_text(
     size_t num_glyphs = 0;
     auto const& rq_glyphs = raqm_get_glyphs(rq, &num_glyphs);
 
-    for (size_t i = 0; i < num_glyphs; i++) {
-        auto const& rglyph = rq_glyphs[i];
+    glyphs.resize(num_glyphs);
+    memcpy(glyphs.data(), rq_glyphs, sizeof(raqm_glyph_t) * num_glyphs);
+}
+
+void FT2Font::set_text(
+    std::u32string_view text, double angle, FT_Int32 flags, std::vector<double> &xys)
+{
+    FT_Matrix matrix; /* transformation matrix */
+
+    angle = angle * (2 * M_PI / 360.0);
+
+    // this computes width and height in subpixels so we have to multiply by 64
+    double cosangle = cos(angle) * 0x10000L;
+    double sinangle = sin(angle) * 0x10000L;
+
+    matrix.xx = (FT_Fixed)cosangle;
+    matrix.xy = (FT_Fixed)-sinangle;
+    matrix.yx = (FT_Fixed)sinangle;
+    matrix.yy = (FT_Fixed)cosangle;
+
+    bbox.xMin = bbox.yMin = 32000;
+    bbox.xMax = bbox.yMax = -32000;
+
+    std::set<FT_String*> glyph_seen_fonts;
+    std::vector<raqm_glyph_t> rq_glyphs;
+    layout(text, flags, glyph_seen_fonts, rq_glyphs);
 
+    for (auto const& rglyph : rq_glyphs) {
         // Warn for missing glyphs.
         if (rglyph.index == 0) {
             ft_glyph_warn(text[rglyph.cluster], glyph_seen_fonts);

src/ft2font.h

@@ -109,6 +109,9 @@ class FT2Font
     void set_size(double ptsize, double dpi);
     void set_charmap(int i);
     void select_charmap(unsigned long i);
+    void layout(std::u32string_view text, FT_Int32 flags,
+                std::set<FT_String*>& glyph_seen_fonts,
+                std::vector<raqm_glyph_t> &glyphs);
     void set_text(std::u32string_view codepoints, double angle, FT_Int32 flags,
                   std::vector<double> &xys);
     int get_kerning(FT_UInt left, FT_UInt right, FT_Kerning_Mode mode);