Improved Path handling

Author: kunitoki

examples/graphics/source/examples/Paths.h

@@ -400,7 +400,7 @@ class PathsExample : public yup::Component
 
         // Parse SVG path data examples - smaller scale
         yup::Path svgHeart;
-        svgHeart.parsePathData ("M12,21.35l-1.45-1.32C5.4,15.36,2,12.28,2,8.5 C2,5.42,4.42,3,7.5,3c1.74,0,3.41,0.81,4.5,2.09C13.09,3.81,14.76,3,16.5,3 C19.58,3,22,5.42,22,8.5c0,3.78-3.4,6.86-8.55,11.54L12,21.35z");
+        svgHeart.fromString ("M12,21.35l-1.45-1.32C5.4,15.36,2,12.28,2,8.5 C2,5.42,4.42,3,7.5,3c1.74,0,3.41,0.81,4.5,2.09C13.09,3.81,14.76,3,16.5,3 C19.58,3,22,5.42,22,8.5c0,3.78-3.4,6.86-8.55,11.54L12,21.35z");
 
         // Scale and position the heart - smaller
         yup::Rectangle<float> heartBounds = svgHeart.getBounds();
@@ -416,7 +416,7 @@ class PathsExample : public yup::Component
 
         // Simple SVG path - smaller
         yup::Path svgTriangle;
-        svgTriangle.parsePathData ("M100,20 L180,160 L20,160 Z");
+        svgTriangle.fromString ("M100,20 L180,160 L20,160 Z");
         svgTriangle.scaleToFit (x + 60, y, 50, 50, true);
 
         g.setFillColor (yup::Color (150, 255, 150));

modules/yup_graphics/primitives/yup_Path.cpp

@@ -733,6 +733,79 @@ rive::RiveRenderPath* Path::getRenderPath() const
     return path.get();
 }
 
+//==============================================================================
+String Path::toString() const
+{
+    const auto& rawPath = path->getRawPath();
+    const auto& points = rawPath.points();
+    const auto& verbs = rawPath.verbs();
+
+    if (points.empty() || verbs.empty())
+        return String();
+
+    String result;
+    result.preallocateBytes (points.size() * 20); // Rough estimate for performance
+
+    size_t pointIndex = 0;
+
+    for (size_t i = 0; i < verbs.size(); ++i)
+    {
+        auto verb = verbs[i];
+
+        switch (verb)
+        {
+            case rive::PathVerb::move:
+                if (pointIndex < points.size())
+                {
+                    result << "M " << points[pointIndex].x << " " << points[pointIndex].y << " ";
+                    pointIndex++;
+                }
+                break;
+
+            case rive::PathVerb::line:
+                if (pointIndex < points.size())
+                {
+                    result << "L " << points[pointIndex].x << " " << points[pointIndex].y << " ";
+                    pointIndex++;
+                }
+                break;
+
+            case rive::PathVerb::quad:
+                // Rive doesn't seem to use quad verbs based on the existing code
+                // But if it does, we'll handle it
+                if (pointIndex + 1 < points.size())
+                {
+                    result << "Q "
+                        << points[pointIndex].x << " " << points[pointIndex].y << " "
+                        << points[pointIndex + 1].x << " " << points[pointIndex + 1].y << " ";
+                    pointIndex += 2;
+                }
+                break;
+
+            case rive::PathVerb::cubic:
+                if (pointIndex + 2 < points.size())
+                {
+                    result << "C "
+                        << points[pointIndex].x << " " << points[pointIndex].y << " "
+                        << points[pointIndex + 1].x << " " << points[pointIndex + 1].y << " "
+                        << points[pointIndex + 2].x << " " << points[pointIndex + 2].y << " ";
+                    pointIndex += 3;
+                }
+                break;
+
+            case rive::PathVerb::close:
+                result << "Z ";
+                break;
+        }
+    }
+
+    // Remove trailing space if present
+    if (result.endsWithChar (' '))
+        result = result.trimEnd();
+
+    return result;
+}
+
 //==============================================================================
 namespace
 {
@@ -1181,7 +1254,7 @@ void handleEllipticalArc (String::CharPointerType& data, Path& path, float& curr
 
 } // namespace
 
-bool Path::parsePathData (const String& pathData)
+bool Path::fromString (const String& pathData)
 {
     // https://dev.w3.org/SVG/tools/svgweb/samples/svg-files/
 

modules/yup_graphics/primitives/yup_Path.h

@@ -583,15 +583,24 @@ class YUP_API Path
     Point<float> getPointAlongPath (float distance) const;
 
     //==============================================================================
+    /** Converts the path to an SVG path data string.
+
+        This method converts the path to a string representation using SVG path data format.
+        The resulting string can be used with fromString() to recreate the path.
+
+        @return A string containing the SVG path data representation of this path.
+    */
+    String toString() const;
+
     /** Parses the path data from a string.
 
-        This method parses the path data from a string and updates the path accordingly.
+        This method parses the path data from a string in SVG path data format and updates the path accordingly.
 
-        @param pathData The string containing the path data.
+        @param pathData The string containing the SVG path data.
 
         @return True if the path data was parsed successfully, false otherwise.
     */
-    bool parsePathData (const String& pathData);
+    bool fromString (const String& pathData);
 
     //==============================================================================
     /** Provides an iterator to the beginning of the path data.

tests/yup_graphics/yup_Path.cpp

@@ -276,16 +276,18 @@ TEST (PathTests, WithRoundedCorners)
     EXPECT_FALSE (same.getBounds().isEmpty());
 }
 
-TEST (PathTests, ParsePathData)
+TEST (PathTests, FromString)
 {
     Path p;
     // Simple SVG path: M10 10 H 90 V 90 H 10 Z
-    bool ok = p.parsePathData ("M10 10 H 90 V 90 H 10 Z");
+    bool ok = p.fromString ("M 10 10 H 90 V 90 H 10 Z");
     EXPECT_TRUE (ok);
     EXPECT_FALSE (p.getBounds().isEmpty());
+    EXPECT_EQ (p.toString(), "M 10 10 L 90 10 L 90 90 L 10 90 Z");
+
     // Edge: malformed path
     Path p2;
-    ok = p2.parsePathData ("M10 10 Q");
+    ok = p2.fromString ("M 10 10 Q");
     EXPECT_TRUE (ok); // Should not throw, but result is empty
 }
 
@@ -455,7 +457,7 @@ TEST (PathTests, AllPublicApiErrorCases)
     p.getPointAlongPath (0.0f);
     p.createStrokePolygon (0.0f);
     p.withRoundedCorners (0.0f);
-    p.parsePathData ("");
+    p.fromString ("");
     SUCCEED();
 }