update

Author: chunhtai

impeller/entity/geometry/stroke_path_geometry.cc

@@ -242,19 +242,22 @@ StrokePathGeometry::CreateSolidStrokeVertices(
   Point offset;
   Point previous_offset;  // Used for computing joins.
 
-  // Computes offset by calculating the direction from point_i - 1 to point_i if point_i
-  // is within `contour_start_point_i` and `contour_end_point_i`; Otherwise, it uses direction
-  // from contour.
-  auto compute_offset = [&polyline, &offset, &previous_offset,
-                         &stroke_width](const size_t point_i, const size_t contour_start_point_i, const size_t contour_end_point_i, const Path::PolylineContour& contour) {
+  // Computes offset by calculating the direction from point_i - 1 to point_i if
+  // point_i is within `contour_start_point_i` and `contour_end_point_i`;
+  // Otherwise, it uses direction from contour.
+  auto compute_offset = [&polyline, &offset, &previous_offset, &stroke_width](
+                            const size_t point_i,
+                            const size_t contour_start_point_i,
+                            const size_t contour_end_point_i,
+                            const Path::PolylineContour& contour) {
     Point direction;
     if (point_i >= contour_end_point_i) {
       direction = contour.end_direction;
     } else if (point_i <= contour_start_point_i) {
-      direction =  - contour.start_direction;
+      direction = -contour.start_direction;
     } else {
       direction =
-        (polyline.points[point_i] - polyline.points[point_i - 1]).Normalize();
+          (polyline.points[point_i] - polyline.points[point_i - 1]).Normalize();
     }
     previous_offset = offset;
     offset = Vector2{-direction.y, direction.x} * stroke_width * 0.5;
@@ -263,7 +266,8 @@ StrokePathGeometry::CreateSolidStrokeVertices(
   auto add_vertices_for_linear_compoent =
       [&vtx_builder, &offset, &previous_offset, &vtx, &polyline,
        &compute_offset, scaled_miter_limit, scale, &join_proc](
-          const size_t component_start_index, const size_t component_end_index, const size_t contour_start_point_i, const size_t contour_end_point_i,
+          const size_t component_start_index, const size_t component_end_index,
+          const size_t contour_start_point_i, const size_t contour_end_point_i,
           const Path::PolylineContour& contour) {
         auto is_last_component =
             component_start_index ==
@@ -275,27 +279,30 @@ StrokePathGeometry::CreateSolidStrokeVertices(
           vtx.position = polyline.points[point_i] + offset;
           vtx_builder.AppendVertex(vtx);
           vtx.position = polyline.points[point_i] - offset;
-          vtx_builder.AppendVertex(vtx); 
+          vtx_builder.AppendVertex(vtx);
 
           // For line components, two additional points need to be appended
           // prior to appending a join connecting the next component.
           vtx.position = polyline.points[point_i + 1] + offset;
           vtx_builder.AppendVertex(vtx);
           vtx.position = polyline.points[point_i + 1] - offset;
           vtx_builder.AppendVertex(vtx);
- 
-          compute_offset(point_i + 2, contour_start_point_i, contour_end_point_i, contour);
+
+          compute_offset(point_i + 2, contour_start_point_i,
+                         contour_end_point_i, contour);
           if (!is_last_component && is_end_of_component) {
             // Generate join from the current line to the next line.
-            join_proc(vtx_builder, polyline.points[point_i + 1], previous_offset,
-                        offset, scaled_miter_limit, scale);
+            join_proc(vtx_builder, polyline.points[point_i + 1],
+                      previous_offset, offset, scaled_miter_limit, scale);
           }
         }
       };
 
   auto add_vertices_for_curve_compoent =
-      [&vtx_builder, &offset, &previous_offset, &vtx, &polyline, &compute_offset, scaled_miter_limit, scale, &join_proc](
-          const size_t component_start_index, const size_t component_end_index, const size_t contour_start_point_i, const size_t contour_end_point_i,
+      [&vtx_builder, &offset, &previous_offset, &vtx, &polyline,
+       &compute_offset, scaled_miter_limit, scale, &join_proc](
+          const size_t component_start_index, const size_t component_end_index,
+          const size_t contour_start_point_i, const size_t contour_end_point_i,
           const Path::PolylineContour& contour) {
         auto is_last_component =
             component_start_index ==
@@ -310,7 +317,8 @@ StrokePathGeometry::CreateSolidStrokeVertices(
           vtx.position = polyline.points[point_i] - offset;
           vtx_builder.AppendVertex(vtx);
 
-          compute_offset(point_i + 2, contour_start_point_i, contour_end_point_i, contour);
+          compute_offset(point_i + 2, contour_start_point_i,
+                         contour_end_point_i, contour);
           // For curve components, the polyline is detailed enough such that
           // it can avoid worrying about joins altogether.
           if (is_end_of_component) {
@@ -320,8 +328,8 @@ StrokePathGeometry::CreateSolidStrokeVertices(
             vtx_builder.AppendVertex(vtx);
             // Generate join from the current line to the next line.
             if (!is_last_component) {
-              join_proc(vtx_builder, polyline.points[point_i + 1], previous_offset,
-                        offset, scaled_miter_limit, scale);
+              join_proc(vtx_builder, polyline.points[point_i + 1],
+                        previous_offset, offset, scaled_miter_limit, scale);
             }
           }
         }
@@ -347,7 +355,8 @@ StrokePathGeometry::CreateSolidStrokeVertices(
         break;
     }
 
-    compute_offset(contour_start_point_i, contour_start_point_i, contour_end_point_i, contour);
+    compute_offset(contour_start_point_i, contour_start_point_i,
+                   contour_end_point_i, contour);
     const Point contour_first_offset = offset;
 
     if (contour_i > 0) {
@@ -394,11 +403,13 @@ StrokePathGeometry::CreateSolidStrokeVertices(
                             : contour.components[contour_component_i + 1]
                                   .component_start_index;
       if (component.is_curve) {
-        add_vertices_for_curve_compoent(component_start_index,
-                                        component_end_index, contour_start_point_i, contour_end_point_i, contour);
+        add_vertices_for_curve_compoent(
+            component_start_index, component_end_index, contour_start_point_i,
+            contour_end_point_i, contour);
       } else {
-        add_vertices_for_linear_compoent(component_start_index,
-                                         component_end_index, contour_start_point_i, contour_end_point_i,contour);
+        add_vertices_for_linear_compoent(
+            component_start_index, component_end_index, contour_start_point_i,
+            contour_end_point_i, contour);
       }
     }