Address some PR comments

Author: bbrk24

Sources/SwiftCrossUI/Path.swift

@@ -299,8 +299,8 @@ public struct Path {
     ///   - endAngle: The angle of the end of the arc, measured in radians clockwise from right.
     ///     Must be between 0 and 2pi (inclusive).
     ///   - clockwise: `true` if the arc is to be drawn clockwise, `false` if the arc is to
-    ///     be drawn counter-clockwise. Used to determine whether to draw the larger arc or
-    ///     the smaller arc identified by the given start and end angles.
+    ///     be drawn counter-clockwise. Used to determine which of the two possible arcs to
+    ///     draw between the given start and end angles.
     public consuming func addArc(
         center: SIMD2<Double>,
         radius: Double,

Sources/SwiftCrossUI/Views/Shapes/Shape.swift

@@ -2,8 +2,7 @@
 ///
 /// If no stroke color or fill color is specified, the default is no stroke and a fill of the
 /// current foreground color.
-public protocol Shape: View
-where Content == EmptyView {
+public protocol Shape: View where Content == EmptyView {
     /// Draw the path for this shape.
     ///
     /// The bounds passed to a shape that is immediately drawn as a view will always have an
@@ -44,8 +43,7 @@ where Content == EmptyView {
     ///   a dry run, while the other properties are used to inform the layout engine how big
     ///   or small the shape can be. The ``ViewSize/idealSize`` property should not vary with
     ///   the `proposal`, and should only depend on the shape's contents. Pass `nil` for the
-    ///   maximum width/height if the shape has no maximum size (and therefore may occupy
-    ///   the entire screen).
+    ///   maximum width/height if the shape has no maximum size.
     func size(fitting proposal: SIMD2<Int>) -> ViewSize
 }
 

Sources/UIKitBackend/UIColor+Color.swift

@@ -30,6 +30,10 @@ extension Color {
 
     var cgColor: CGColor {
         CGColor(
-            red: CGFloat(red), green: CGFloat(green), blue: CGFloat(blue), alpha: CGFloat(alpha))
+            red: CGFloat(red),
+            green: CGFloat(green),
+            blue: CGFloat(blue),
+            alpha: CGFloat(alpha)
+        )
     }
 }

Sources/UIKitBackend/UIKitBackend+Path.swift

@@ -129,25 +129,29 @@ extension UIKitBackend {
         shapeLayer.miterLimit = path.miterLimit
         shapeLayer.fillRule = path.usesEvenOddFillRule ? .evenOdd : .nonZero
 
-        shapeLayer.lineJoin =
-            switch path.lineJoinStyle {
-                case .miter:
-                    .miter
-                case .round:
-                    .round
-                case .bevel:
-                    .bevel
-            }
+        switch path.lineJoinStyle {
+            case .miter:
+                shapeLayer.lineJoin = .miter
+            case .round:
+                shapeLayer.lineJoin = .round
+            case .bevel:
+                shapeLayer.lineJoin = .bevel
+            @unknown default:
+                print("Warning: unrecognized lineJoinStyle \(path.lineJoinStyle)")
+                shapeLayer.lineJoin = .miter
+        }
 
-        shapeLayer.lineCap =
-            switch path.lineCapStyle {
-                case .butt:
-                    .butt
-                case .round:
-                    .round
-                case .square:
-                    .square
-            }
+        switch path.lineCapStyle {
+            case .butt:
+                shapeLayer.lineCap = .butt
+            case .round:
+                shapeLayer.lineCap = .round
+            case .square:
+                shapeLayer.lineCap = .square
+            @unknown default:
+                print("Warning: unrecognized lineCapStyle \(path.lineCapStyle)")
+                shapeLayer.lineCap = .butt
+        }
 
         shapeLayer.strokeColor = strokeColor.cgColor
         shapeLayer.fillColor = fillColor.cgColor

Sources/WinUIBackend/WinUIBackend.swift

@@ -1191,26 +1191,26 @@ public final class WinUIBackend: AppBackend {
         _ collection: WinUI.GeometryCollection,
         lastPoint: Point
     ) -> PathFigure {
-        var pathGeo: PathGeometry
+        var pathGeometry: PathGeometry
         if collection.size > 0,
             let castedLast = collection.getAt(collection.size - 1) as? PathGeometry
         {
-            pathGeo = castedLast
+            pathGeometry = castedLast
         } else {
-            pathGeo = PathGeometry()
-            collection.append(pathGeo)
+            pathGeometry = PathGeometry()
+            collection.append(pathGeometry)
         }
 
         var figure: PathFigure
-        if pathGeo.figures.size > 0 {
+        if pathGeometry.figures.size > 0 {
             // Note: the if check and force-unwrap is necessary. You can't do an `if let`
             // here because PathFigureCollection uses unsigned integers for its indices so
             // `size - 1` would underflow (causing a fatalError) if it's empty.
-            figure = pathGeo.figures.getAt(pathGeo.figures.size - 1)!
+            figure = pathGeometry.figures.getAt(pathGeometry.figures.size - 1)!
         } else {
             figure = PathFigure()
             figure.startPoint = lastPoint
-            pathGeo.figures.append(figure)
+            pathGeometry.figures.append(figure)
         }
 
         return figure
@@ -1226,14 +1226,14 @@ public final class WinUIBackend: AppBackend {
                     lastPoint = Point(x: Float(point.x), y: Float(point.y))
 
                     if geometry.size > 0,
-                        let pathGeo = geometry.getAt(geometry.size - 1) as? PathGeometry,
-                        pathGeo.figures.size > 0
+                        let pathGeometry = geometry.getAt(geometry.size - 1) as? PathGeometry,
+                        pathGeometry.figures.size > 0
                     {
-                        let figure = pathGeo.figures.getAt(pathGeo.figures.size - 1)!
+                        let figure = pathGeometry.figures.getAt(pathGeometry.figures.size - 1)!
                         if figure.segments.size > 0 {
                             let newFigure = PathFigure()
                             newFigure.startPoint = lastPoint
-                            pathGeo.figures.append(newFigure)
+                            pathGeometry.figures.append(newFigure)
                         } else {
                             figure.startPoint = lastPoint
                         }
@@ -1362,8 +1362,8 @@ public final class WinUIBackend: AppBackend {
                     }
 
                     if geometry.size > 0,
-                        let pathGeo = geometry.getAt(geometry.size - 1) as? PathGeometry,
-                        pathGeo.figures.contains(where: { ($0?.segments.size ?? 0) > 0 })
+                        let pathGeometry = geometry.getAt(geometry.size - 1) as? PathGeometry,
+                        pathGeometry.figures.contains(where: { ($0?.segments.size ?? 0) > 0 })
                     {
                         // Start a new PathGeometry so that transforms don't apply going forward
                         geometry.append(PathGeometry())