[pango/kill-ft-face: 23/33] fc: Make pangofc_shape backend-neutral

[Matthias Clasen <matthiasc src gnome org>]

commit c01cbc25b884d946602eb06ea1934d5710dd60da
Author: Matthias Clasen <mclasen redhat com>
Date:   Mon Nov 19 13:59:26 2018 -0500

    fc: Make pangofc_shape backend-neutral
    
    Use typechecks for the two remaining bits of
    code that use PangoFcFont.

 pango/pangofc-shape.c | 112 ++++++++++++++++++++++----------------------------
 1 file changed, 49 insertions(+), 63 deletions(-)
---
diff --git a/pango/pangofc-shape.c b/pango/pangofc-shape.c
index 9ea9c25f..04a961d3 100644
--- a/pango/pangofc-shape.c
+++ b/pango/pangofc-shape.c
@@ -79,7 +79,6 @@ _pango_fc_shape (PangoFont           *font,
                 const char          *paragraph_text,
                 unsigned int         paragraph_length)
 {
-  PangoFcFont *fc_font;
   hb_font_t *hb_font;
   hb_buffer_t *hb_buffer;
   hb_direction_t hb_direction;
@@ -118,24 +117,7 @@ _pango_fc_shape (PangoFont           *font,
 
   hb_buffer_add_utf8 (hb_buffer, paragraph_text, paragraph_length, item_offset, item_length);
 
-  /* Setup features from fontconfig pattern. */
-  fc_font = PANGO_FC_FONT (font);
-  if (fc_font->font_pattern)
-    {
-      char *s;
-      while (num_features < G_N_ELEMENTS (features) &&
-            FcResultMatch == FcPatternGetString (fc_font->font_pattern,
-                                                 PANGO_FC_FONT_FEATURES,
-                                                 num_features,
-                                                 (FcChar8 **) &s))
-       {
-         gboolean ret = hb_feature_from_string (s, -1, &features[num_features]);
-         features[num_features].start = 0;
-         features[num_features].end   = (unsigned int) -1;
-         if (ret)
-           num_features++;
-       }
-    }
+  pango_font_get_features (font, features, G_N_ELEMENTS (features), &num_features);
 
   if (analysis->extra_attrs)
     {
@@ -151,8 +133,7 @@ _pango_fc_shape (PangoFont           *font,
              const gchar *end;
              int len;
 
-             feat = fattr->features;
-
+              feat = fattr->features;
              while (feat != NULL && num_features < G_N_ELEMENTS (features))
                 {
                   end = strchr (feat, ',');
@@ -160,13 +141,14 @@ _pango_fc_shape (PangoFont           *font,
                     len = end - feat;
                   else
                     len = -1;
+
                   if (hb_feature_from_string (feat, len, &features[num_features]))
                     {
                       features[num_features].start = attr->start_index;
                       features[num_features].end = attr->end_index;
                       num_features++;
                     }
-                    
+
                   if (end == NULL)
                     break;
 
@@ -215,57 +197,61 @@ _pango_fc_shape (PangoFont           *font,
        hb_position++;
       }
 
-  if (fc_font->is_hinted)
-  {
-    double x_scale_inv, y_scale_inv;
-    double x_scale, y_scale;
-    PangoFcFontKey *key;
+  if (PANGO_IS_FC_FONT (font))
+    {
+      PangoFcFont *fc_font = PANGO_FC_FONT (font);
+      if (fc_font->is_hinted)
+        {
+          double x_scale_inv, y_scale_inv;
+          double x_scale, y_scale;
+          PangoFcFontKey *key;
 
-    x_scale_inv = y_scale_inv = 1.0;
-    key = _pango_fc_font_get_font_key (fc_font);
-    if (key)
-      {
-        const PangoMatrix *matrix = pango_fc_font_key_get_matrix (key);
-        pango_matrix_get_font_scale_factors (matrix, &x_scale_inv, &y_scale_inv);
-      }
-    if (PANGO_GRAVITY_IS_IMPROPER (analysis->gravity))
-      {
-        x_scale_inv = -x_scale_inv;
-        y_scale_inv = -y_scale_inv;
-      }
-    x_scale = 1. / x_scale_inv;
-    y_scale = 1. / y_scale_inv;
+          x_scale_inv = y_scale_inv = 1.0;
+          key = _pango_fc_font_get_font_key (fc_font);
+          if (key)
+            {
+              const PangoMatrix *matrix = pango_fc_font_key_get_matrix (key);
+              pango_matrix_get_font_scale_factors (matrix, &x_scale_inv, &y_scale_inv);
+            }
+          if (PANGO_GRAVITY_IS_IMPROPER (analysis->gravity))
+            {
+              x_scale_inv = -x_scale_inv;
+              y_scale_inv = -y_scale_inv;
+            }
+          x_scale = 1. / x_scale_inv;
+          y_scale = 1. / y_scale_inv;
 
-    if (x_scale == 1.0 && y_scale == 1.0)
-      {
-       for (i = 0; i < num_glyphs; i++)
-         infos[i].geometry.width = PANGO_UNITS_ROUND (infos[i].geometry.width);
-      }
-    else
-      {
+          if (x_scale == 1.0 && y_scale == 1.0)
+            {
+              for (i = 0; i < num_glyphs; i++)
+                infos[i].geometry.width = PANGO_UNITS_ROUND (infos[i].geometry.width);
+            }
+          else
+            {
 #if 0
-       if (PANGO_GRAVITY_IS_VERTICAL (analysis->gravity))
-         {
-           /* XXX */
-           double tmp = x_scale;
-           x_scale = y_scale;
-           y_scale = -tmp;
-         }
+              if (PANGO_GRAVITY_IS_VERTICAL (analysis->gravity))
+                {
+                  /* XXX */
+                  double tmp = x_scale;
+                  x_scale = y_scale;
+                  y_scale = -tmp;
+                }
 #endif
 #define HINT(value, scale_inv, scale) (PANGO_UNITS_ROUND ((int) ((value) * scale)) * scale_inv)
 #define HINT_X(value) HINT ((value), x_scale, x_scale_inv)
 #define HINT_Y(value) HINT ((value), y_scale, y_scale_inv)
-       for (i = 0; i < num_glyphs; i++)
-         {
-           infos[i].geometry.width    = HINT_X (infos[i].geometry.width);
-           infos[i].geometry.x_offset = HINT_X (infos[i].geometry.x_offset);
-           infos[i].geometry.y_offset = HINT_Y (infos[i].geometry.y_offset);
-         }
+              for (i = 0; i < num_glyphs; i++)
+                {
+                  infos[i].geometry.width    = HINT_X (infos[i].geometry.width);
+                  infos[i].geometry.x_offset = HINT_X (infos[i].geometry.x_offset);
+                  infos[i].geometry.y_offset = HINT_Y (infos[i].geometry.y_offset);
+                }
 #undef HINT_Y
 #undef HINT_X
 #undef HINT
-      }
-  }
+            }
+        }
+    }
 
   release_buffer (hb_buffer, free_buffer);
 }