[gtk/wip/otte/lottie: 2432/2503] path: Implement gsk_path_to_cairo() using foreach()

[Benjamin Otte <otte src gnome org>]

commit bb760ce1ed9378d6d384a683479f9cc7d69e7cd6
Author: Benjamin Otte <otte redhat com>
Date:   Wed Nov 18 05:22:21 2020 +0100

    path: Implement gsk_path_to_cairo() using foreach()

 gsk/gskpath.c        | 110 ++++++++++++++++++++++-----------------------------
 gsk/gskpathprivate.h |   4 ++
 2 files changed, 52 insertions(+), 62 deletions(-)
---
diff --git a/gsk/gskpath.c b/gsk/gskpath.c
index c7b1b1a285..0d295dc594 100644
--- a/gsk/gskpath.c
+++ b/gsk/gskpath.c
@@ -58,8 +58,6 @@ struct _GskContourClass
   GskPathFlags          (* get_flags)           (const GskContour       *contour);
   void                  (* print)               (const GskContour       *contour,
                                                  GString                *string);
-  void                  (* to_cairo)            (const GskContour       *contour,
-                                                 cairo_t                *cr);
   gboolean              (* get_bounds)          (const GskContour       *contour,
                                                  graphene_rect_t        *bounds);
   gboolean              (* foreach)             (const GskContour       *contour,
@@ -163,17 +161,6 @@ gsk_rect_contour_print (const GskContour *contour,
   g_string_append (string, " z");
 }
 
-static void
-gsk_rect_contour_to_cairo (const GskContour *contour,
-                           cairo_t          *cr)
-{
-  const GskRectContour *self = (const GskRectContour *) contour;
-
-  cairo_rectangle (cr,
-                   self->x, self->y,
-                   self->width, self->height);
-}
-
 static gboolean
 gsk_rect_contour_get_bounds (const GskContour *contour,
                              graphene_rect_t  *rect)
@@ -307,7 +294,6 @@ static const GskContourClass GSK_RECT_CONTOUR_CLASS =
   gsk_contour_get_size_default,
   gsk_rect_contour_get_flags,
   gsk_rect_contour_print,
-  gsk_rect_contour_to_cairo,
   gsk_rect_contour_get_bounds,
   gsk_rect_contour_foreach,
   gsk_rect_contour_init_measure,
@@ -446,44 +432,6 @@ gsk_standard_contour_print (const GskContour *contour,
     }
 }
 
-static void
-gsk_standard_contour_to_cairo (const GskContour *contour,
-                               cairo_t          *cr)
-{
-  const GskStandardContour *self = (const GskStandardContour *) contour;
-  gsize i;
-
-  cairo_new_sub_path (cr);
-
-  for (i = 0; i < self->n_ops; i ++)
-    {
-      graphene_point_t *pt = &self->points[self->ops[i].point];
-
-      switch (self->ops[i].op)
-      {
-        case GSK_PATH_MOVE:
-          cairo_move_to (cr, pt[0].x, pt[0].y);
-          break;
-
-        case GSK_PATH_CLOSE:
-          cairo_close_path (cr);
-          break;
-
-        case GSK_PATH_LINE:
-          cairo_line_to (cr, pt[1].x, pt[1].y);
-          break;
-
-        case GSK_PATH_CURVE:
-          cairo_curve_to (cr, pt[1].x, pt[1].y, pt[2].x, pt[2].y, pt[3].x, pt[3].y);
-          break;
-
-        default:
-          g_assert_not_reached();
-          return;
-      }
-    }
-}
-
 static void
 rect_add_point (graphene_rect_t        *rect,
                 const graphene_point_t *point)
@@ -697,7 +645,6 @@ static const GskContourClass GSK_STANDARD_CONTOUR_CLASS =
   gsk_standard_contour_get_size,
   gsk_standard_contour_get_flags,
   gsk_standard_contour_print,
-  gsk_standard_contour_to_cairo,
   gsk_standard_contour_get_bounds,
   gsk_standard_contour_foreach,
   gsk_standard_contour_init_measure,
@@ -948,6 +895,38 @@ gsk_path_to_string (GskPath *self)
   return g_string_free (string, FALSE);
 }
 
+static gboolean
+gsk_path_to_cairo_add_op (GskPathOperation        op,
+                          const graphene_point_t *pts,
+                          gsize                   n_pts,
+                          gpointer                cr)
+{
+  switch (op)
+  {
+    case GSK_PATH_MOVE:
+      cairo_move_to (cr, pts[0].x, pts[0].y);
+      break;
+
+    case GSK_PATH_CLOSE:
+      cairo_close_path (cr);
+      break;
+
+    case GSK_PATH_LINE:
+      cairo_line_to (cr, pts[1].x, pts[1].y);
+      break;
+
+    case GSK_PATH_CURVE:
+      cairo_curve_to (cr, pts[1].x, pts[1].y, pts[2].x, pts[2].y, pts[3].x, pts[3].y);
+      break;
+
+    default:
+      g_assert_not_reached ();
+      return FALSE;
+  }
+
+  return TRUE;
+}
+
 /**
  * gsk_path_to_cairo:
  * @self: a #GskPath
@@ -966,15 +945,13 @@ void
 gsk_path_to_cairo (GskPath *self,
                    cairo_t *cr)
 {
-  gsize i;
-
   g_return_if_fail (self != NULL);
   g_return_if_fail (cr != NULL);
 
-  for (i = 0; i < self->n_contours; i++)
-    {
-      self->contours[i]->klass->to_cairo (self->contours[i], cr);
-    }
+  gsk_path_foreach_with_tolerance (self,
+                                   cairo_get_tolerance (cr),
+                                   gsk_path_to_cairo_add_op,
+                                   cr);
 }
 
 /*
@@ -1078,14 +1055,23 @@ gsk_path_foreach (GskPath            *self,
                   GskPathForeachFunc  func,
                   gpointer            user_data)
 {
-  gsize i;
-
   g_return_val_if_fail (self != NULL, FALSE);
   g_return_val_if_fail (func, FALSE);
 
+  return gsk_path_foreach_with_tolerance (self, GSK_PATH_TOLERANCE_DEFAULT, func, user_data);
+}
+
+gboolean
+gsk_path_foreach_with_tolerance (GskPath            *self,
+                                 double              tolerance,
+                                 GskPathForeachFunc  func,
+                                 gpointer            user_data)
+{
+  gsize i;
+
   for (i = 0; i < self->n_contours; i++)
     {
-      if (!gsk_contour_foreach (self->contours[i], GSK_PATH_TOLERANCE_DEFAULT, func, user_data))
+      if (!gsk_contour_foreach (self->contours[i], tolerance, func, user_data))
         return FALSE;
     }
 
diff --git a/gsk/gskpathprivate.h b/gsk/gskpathprivate.h
index 2e47ac465f..afee649ce1 100644
--- a/gsk/gskpathprivate.h
+++ b/gsk/gskpathprivate.h
@@ -29,6 +29,10 @@ G_BEGIN_DECLS
 #define GSK_PATH_TOLERANCE_DEFAULT (0.1)
 
 gsize                   gsk_path_get_n_contours                 (GskPath              *path);
+gboolean                gsk_path_foreach_with_tolerance         (GskPath              *self,
+                                                                 double                tolerance,
+                                                                 GskPathForeachFunc    func,
+                                                                 gpointer              user_data);
 
 gpointer                gsk_contour_init_measure                (GskPath              *path,
                                                                  gsize                 i,