DOM elements are not actually Sendable

Author: swhitty

DOM/Sources/DOM.Element.swift

@@ -44,15 +44,15 @@ package extension DOM {
 
     class Element {}
 
-    class GraphicsElement: Element, ElementAttributes, @unchecked Sendable {
+    class GraphicsElement: Element, ElementAttributes {
         package var id: String?
         package var `class`: String?
 
         package var attributes = PresentationAttributes()
         package var style = PresentationAttributes()
     }
 
-    final class Line: GraphicsElement, @unchecked Sendable {
+    final class Line: GraphicsElement {
         package var x1: Coordinate
         package var y1: Coordinate
         package var x2: Coordinate
@@ -67,7 +67,7 @@ package extension DOM {
         }
     }
 
-    final class Circle: GraphicsElement, @unchecked Sendable {
+    final class Circle: GraphicsElement {
         package var cx: Coordinate?
         package var cy: Coordinate?
         package var r: Coordinate
@@ -80,7 +80,7 @@ package extension DOM {
         }
     }
 
-    final class Ellipse: GraphicsElement, @unchecked Sendable {
+    final class Ellipse: GraphicsElement {
         package var cx: Coordinate?
         package var cy: Coordinate?
         package var rx: Coordinate
@@ -95,7 +95,7 @@ package extension DOM {
         }
     }
 
-    final class Rect: GraphicsElement, @unchecked Sendable {
+    final class Rect: GraphicsElement {
         package var x: Coordinate?
         package var y: Coordinate?
         package var width: Coordinate
@@ -113,7 +113,7 @@ package extension DOM {
         }
     }
 
-    final class Polyline: GraphicsElement, @unchecked Sendable {
+    final class Polyline: GraphicsElement {
         package var points: [Point]
 
         package init(points: [Point]) {
@@ -122,7 +122,7 @@ package extension DOM {
         }
     }
 
-    final class Polygon: GraphicsElement, @unchecked Sendable {
+    final class Polygon: GraphicsElement {
         package var points: [Point]
 
         package init(points: [Point]) {
@@ -131,7 +131,7 @@ package extension DOM {
         }
     }
 
-    final class Group: GraphicsElement, ContainerElement, @unchecked Sendable {
+    final class Group: GraphicsElement, ContainerElement {
         package var childElements = [GraphicsElement]()
     }
 }

DOM/Sources/DOM.Image.swift

@@ -29,7 +29,7 @@
 //  3. This notice may not be removed or altered from any source distribution.
 //
 package extension DOM {
-    final class Image: GraphicsElement, @unchecked Sendable {
+    final class Image: GraphicsElement {
         package var href: URL
         package var width: Coordinate?
         package var height: Coordinate?

DOM/Sources/DOM.Path.swift

@@ -33,7 +33,7 @@ import Foundation
 
 package extension DOM {
 
-    final class Path: GraphicsElement, @unchecked Sendable {
+    final class Path: GraphicsElement {
 
         // segments[0] should always be a .move
         package var segments: [Segment]

DOM/Sources/DOM.SVG.swift

@@ -30,7 +30,7 @@
 //
 
 package extension DOM {
-    final class SVG: GraphicsElement, ContainerElement, @unchecked Sendable {
+    final class SVG: GraphicsElement, ContainerElement {
         package var x: Coordinate?
         package var y: Coordinate?
         package var width: Length

DOM/Sources/DOM.Switch.swift

@@ -30,7 +30,7 @@
 //
 
 package extension DOM {
-    final class Switch: GraphicsElement, ContainerElement, @unchecked Sendable {
+    final class Switch: GraphicsElement, ContainerElement {
         package var childElements = [DOM.GraphicsElement]()
     }
 }

DOM/Sources/DOM.Text.swift

@@ -33,7 +33,7 @@ import Foundation
 
 package extension DOM {
 
-    final class Text: GraphicsElement, @unchecked Sendable {
+    final class Text: GraphicsElement {
         package var x: Coordinate?
         package var y: Coordinate?
         package var value: String
@@ -45,7 +45,7 @@ package extension DOM {
         }
     }
 
-    final class Anchor: GraphicsElement, ContainerElement, @unchecked Sendable {
+    final class Anchor: GraphicsElement, ContainerElement {
         package var href: URL?
         package var childElements = [GraphicsElement]()
     }

DOM/Sources/DOM.Use.swift

@@ -30,7 +30,7 @@
 //
 
 package extension DOM {
-    final class Use: GraphicsElement, @unchecked Sendable {
+    final class Use: GraphicsElement {
         package var x: Coordinate?
         package var y: Coordinate?
 

SwiftDraw/XML.Formatter.SVG.swift

@@ -35,7 +35,7 @@ import Foundation
 extension XML.Formatter {
 
     enum Error: Swift.Error {
-        case unsupportedGraphicsElement(DOM.GraphicsElement)
+        case unsupportedGraphicsElement(name: String)
     }
 
     struct SVG {
@@ -198,7 +198,7 @@ extension XML.Formatter {
             } else if let path = graphic as? DOM.Path {
                 element = makeElement(from: path)
             } else {
-                throw Error.unsupportedGraphicsElement(graphic)
+                throw Error.unsupportedGraphicsElement(name: String(describing: type(of: graphic)))
             }
 
             if let container = graphic as? any ContainerElement {