Successive explicit MoveTo commands should be treated as MoveTo.

Author: luostari

SwiftSVG/SVG/Iterators/PathCommand.swift

@@ -69,7 +69,7 @@ internal protocol PathCommand: PreviousCommand {
      - Parameter path: The path to append a new path to
      - Parameter previousCommand: An optional previous command. Used primarily with the shortcut cubic and quadratic Bezier types
      */
-    func execute(on path: UIBezierPath, previousCommand: PreviousCommand?)
+    func execute(on path: UIBezierPath, previousCommand: PreviousCommand?) -> PreviousCommand?
 }
 
 /**
@@ -162,25 +162,24 @@ internal struct MoveTo: PathCommand {
     /**
      This will move the current point to `CGPoint(self.coordinateBuffer[0], self.coordinateBuffer[1])`.
 
-     Sequential MoveTo commands should be treated as LineTos.
+     Sequential implicit MoveTo commands should be treated as LineTos.
 
      From Docs (https://www.w3.org/TR/SVG2/paths.html#PathDataMovetoCommands):
 
      ```
      Start a new sub-path at the given (x,y) coordinates. M (uppercase) indicates that absolute coordinates will follow; m (lowercase) indicates that relative coordinates will follow. If a moveto is followed by multiple pairs of coordinates, the subsequent pairs are treated as implicit lineto commands. Hence, implicit lineto commands will be relative if the moveto is relative, and absolute if the moveto is absolute.
      ```
      */
-    internal func execute(on path: UIBezierPath, previousCommand: PreviousCommand? = nil) {
-        
+    internal func execute(on path: UIBezierPath, previousCommand: PreviousCommand? = nil) -> PreviousCommand? {
         if previousCommand is MoveTo {
             var implicitLineTo = LineTo(pathType: self.pathType)
             implicitLineTo.coordinateBuffer = [self.coordinateBuffer[0], self.coordinateBuffer[1]]
-            implicitLineTo.execute(on: path)
-            return
+            return implicitLineTo.execute(on: path)
         }
 
         let point = self.pointForPathType(CGPoint(x: self.coordinateBuffer[0], y: self.coordinateBuffer[1]), relativeTo: path.currentPoint)
         path.move(to: point)
+        return self
     }
 }
 
@@ -206,8 +205,9 @@ internal struct ClosePath: PathCommand {
     /**
      Closes the current path
      */
-    internal func execute(on path: UIBezierPath, previousCommand: PreviousCommand? = nil) {
+    internal func execute(on path: UIBezierPath, previousCommand: PreviousCommand? = nil) -> PreviousCommand? {
         path.close()
+        return self
     }
 
 }
@@ -234,9 +234,10 @@ internal struct LineTo: PathCommand {
     /**
      Creates a line from the `path.currentPoint` to point `CGPoint(self.coordinateBuffer[0], coordinateBuffer[1])`
      */
-    internal func execute(on path: UIBezierPath, previousCommand: PreviousCommand? = nil) {
+    internal func execute(on path: UIBezierPath, previousCommand: PreviousCommand? = nil) -> PreviousCommand? {
         let point = self.pointForPathType(CGPoint(x: self.coordinateBuffer[0], y: self.coordinateBuffer[1]), relativeTo: path.currentPoint)
         path.addLine(to: point)
+        return self
     }
 }
 
@@ -262,10 +263,11 @@ internal struct HorizontalLineTo: PathCommand {
     /**
      Adds a horizontal line from the currentPoint to `CGPoint(self.coordinateBuffer[0], path.currentPoint.y)`
      */
-    internal func execute(on path: UIBezierPath, previousCommand: PreviousCommand? = nil) {
+    internal func execute(on path: UIBezierPath, previousCommand: PreviousCommand? = nil) -> PreviousCommand? {
         let x = self.coordinateBuffer[0]
         let point = (self.pathType == .absolute ? CGPoint(x: CGFloat(x), y: path.currentPoint.y) : CGPoint(x: path.currentPoint.x + CGFloat(x), y: path.currentPoint.y))
         path.addLine(to: point)
+        return self
     }
 }
 
@@ -291,10 +293,11 @@ internal struct VerticalLineTo: PathCommand {
     /**
      Adds a vertical line from the currentPoint to `CGPoint(path.currentPoint.y, self.coordinateBuffer[0])`
      */
-    internal func execute(on path: UIBezierPath, previousCommand: PreviousCommand? = nil) {
+    internal func execute(on path: UIBezierPath, previousCommand: PreviousCommand? = nil) -> PreviousCommand? {
         let y = self.coordinateBuffer[0]
         let point = (self.pathType == .absolute ? CGPoint(x: path.currentPoint.x, y: CGFloat(y)) : CGPoint(x: path.currentPoint.x, y: path.currentPoint.y + CGFloat(y)))
         path.addLine(to: point)
+        return self
     }
 }
 
@@ -320,11 +323,12 @@ internal struct CurveTo: PathCommand {
     /**
      Adds a cubic Bezier curve to `path`. The path will end up at `CGPoint(self.coordinateBuffer[4], self.coordinateBuffer[5])`. The control point for `path.currentPoint` will be `CGPoint(self.coordinateBuffer[0], self.coordinateBuffer[1])`. Then controle point for the end point will be CGPoint(self.coordinateBuffer[2], self.coordinateBuffer[3])
      */
-    internal func execute(on path: UIBezierPath, previousCommand: PreviousCommand? = nil) {
+    internal func execute(on path: UIBezierPath, previousCommand: PreviousCommand? = nil) -> PreviousCommand? {
         let startControl = self.pointForPathType(CGPoint(x: self.coordinateBuffer[0], y: self.coordinateBuffer[1]), relativeTo: path.currentPoint)
         let endControl = self.pointForPathType(CGPoint(x: self.coordinateBuffer[2], y: self.coordinateBuffer[3]), relativeTo: path.currentPoint)
         let point = self.pointForPathType(CGPoint(x: self.coordinateBuffer[4], y: self.coordinateBuffer[5]), relativeTo: path.currentPoint)
         path.addCurve(to: point, controlPoint1: startControl, controlPoint2: endControl)
+        return self
     }
 }
 
@@ -350,7 +354,7 @@ internal struct SmoothCurveTo: PathCommand {
     /**
      Shortcut cubic Bezier curve to that add a new path ending up at `CGPoint(self.coordinateBuffer[0], self.coordinateBuffer[1])` with a single control point in the middle.
      */
-    internal func execute(on path: UIBezierPath, previousCommand: PreviousCommand? = nil) {
+    internal func execute(on path: UIBezierPath, previousCommand: PreviousCommand? = nil) -> PreviousCommand? {
 
         let point = self.pointForPathType(CGPoint(x: self.coordinateBuffer[2], y: self.coordinateBuffer[3]), relativeTo: path.currentPoint)
         let controlEnd = self.pointForPathType(CGPoint(x: self.coordinateBuffer[0], y: self.coordinateBuffer[1]), relativeTo: path.currentPoint)
@@ -394,6 +398,7 @@ internal struct SmoothCurveTo: PathCommand {
             controlStart = path.currentPoint
         }
         path.addCurve(to: point, controlPoint1: controlStart, controlPoint2: controlEnd)
+        return self
     }
 }
 
@@ -416,10 +421,11 @@ internal struct QuadraticCurveTo: PathCommand {
         self.pathType = pathType
     }
 
-    internal func execute(on path: UIBezierPath, previousCommand: PreviousCommand? = nil) {
+    internal func execute(on path: UIBezierPath, previousCommand: PreviousCommand? = nil) -> PreviousCommand? {
         let controlPoint = self.pointForPathType(CGPoint(x: self.coordinateBuffer[0], y: self.coordinateBuffer[1]), relativeTo: path.currentPoint)
         let point = self.pointForPathType(CGPoint(x: self.coordinateBuffer[2], y: self.coordinateBuffer[3]), relativeTo: path.currentPoint)
         path.addQuadCurve(to: point, controlPoint: controlPoint)
+        return self
     }
 }
 
@@ -445,7 +451,7 @@ internal struct SmoothQuadraticCurveTo: PathCommand {
         self.pathType = pathType
     }
 
-    internal func execute(on path: UIBezierPath, previousCommand: PreviousCommand? = nil) {
+    internal func execute(on path: UIBezierPath, previousCommand: PreviousCommand? = nil) -> PreviousCommand? {
 
         let point = self.pointForPathType(CGPoint(x: self.coordinateBuffer[0], y: self.coordinateBuffer[1]), relativeTo: path.currentPoint)
 
@@ -471,6 +477,7 @@ internal struct SmoothQuadraticCurveTo: PathCommand {
             controlPoint = path.currentPoint
         }
         path.addQuadCurve(to: point, controlPoint: controlPoint)
+        return self
     }
 }
 
@@ -495,7 +502,8 @@ internal struct EllipticalArc: PathCommand {
     }
 
     /// :nodoc:
-    internal func execute(on path: UIBezierPath, previousCommand: PreviousCommand? = nil) {
+    internal func execute(on path: UIBezierPath, previousCommand: PreviousCommand? = nil) -> PreviousCommand? {
         assert(false, "Needs Implementation")
+        return self
     }
 }