Merge pull request rarestype#122 from rarestype/optimize-string-rendering

[patch]: optimize string descriptions

Author: tayloraswift

Sources/JSONAST/JSON.Array.swift

@@ -16,7 +16,7 @@ extension JSON.Array: CustomStringConvertible {
     /// Reparsing and reserializing this string is guaranteed to return the
     /// same string.
     public var description: String {
-        "[\(self.elements.map(\.description).joined(separator: ","))]"
+        "[\(self.elements.lazy.map(\.description).joined(separator: ","))]"
     }
 }
 extension JSON.Array: ExpressibleByArrayLiteral {

Sources/JSONAST/JSON.Key.swift

@@ -15,6 +15,9 @@ extension JSON.Key {
         self.init(rawValue: codingKey.stringValue)
     }
 }
+extension JSON.Key {
+    var quoted: JSON.Literal<String> { .init(self.rawValue) }
+}
 extension JSON.Key: CustomStringConvertible {
     @inlinable public var description: String {
         self.rawValue

Sources/JSONAST/JSON.Number.Base10.Inverse.swift

@@ -6,8 +6,7 @@ extension JSON.Number.Base10 {
         ///     - x: A positive exponent. If `x` is `2`, this subscript
         ///         will return `1e-2`.
         ///     - _: A ``BinaryFloatingPoint`` type.
-        static subscript<T>(x: Int, as _: T.Type) -> T
-            where T: BinaryFloatingPoint {
+        static subscript<T>(x: Int, as _: T.Type) -> T where T: BinaryFloatingPoint {
             let inverses: [T] = [
                 1,
                 1e-1,

Sources/JSONAST/JSON.Number.Inline.swift

@@ -31,9 +31,9 @@ extension JSON.Number {
         ///     - units: The magnitude, in units of `places`.
         ///     - places: The number of decimal places.
         @inlinable public init(sign: FloatingPointSign, units: UInt64, places: UInt32 = 0) {
-            self.sign       = sign
-            self.units      = units
-            self._places    = places
+            self.sign = sign
+            self.units = units
+            self._places = places
         }
     }
 }
@@ -138,9 +138,9 @@ extension JSON.Number.Inline: CustomStringConvertible {
             case .minus: return "-\(self.units)"
             }
         }
-        let places: Int      = .init(self.places)
         let unpadded: String = .init(self.units)
-        let string: String   = """
+        let places: Int = .init(self.places)
+        let string: String = """
         \(String.init(repeating: "0", count: Swift.max(0, 1 + places - unpadded.count)))\
         \(unpadded)
         """

Sources/JSONAST/JSON.Object.swift

@@ -39,9 +39,11 @@ extension JSON.Object: CustomStringConvertible {
     /// same string.
     public var description: String {
         """
-        {\(self.fields.map {
-                "\(String.init(JSON.Literal<String>.init($0.key.rawValue))):\($0.value)"
-            }.joined(separator: ","))}
+        {\(
+            self.fields.lazy.map {
+                "\(String.init($0.key.quoted)):\($0.value)"
+            }.joined(separator: ",")
+        )}
         """
     }
 }

Sources/JSONAST/JSON.TypecastError.swift

@@ -10,7 +10,7 @@ extension JSON {
     }
 }
 extension JSON.TypecastError {
-    @inlinable public init(invalid json: __shared JSON.Node) {
+    @inlinable public init(invalid json: borrowing JSON.Node) {
         switch json {
         case .null: self = .null
         case .bool: self = .bool