Following up on closures property wrappers cleanup (#60)

This change fixes an issue with the use of property wrappers for closures in WebAPIKit, which caused unnecessary increases in binary code size for products utilizing this library. Use of property wrappers is removed and simple direct getter and setter accessors are used for properties of closure types instead.

Author: MaxDesiatov

Sources/WebIDLToSwift/WebIDL+SwiftRepresentation.swift

@@ -36,47 +36,27 @@ extension IDLAttribute: SwiftRepresentable {
         }
         if ModuleState.override {
             assert(!ModuleState.static)
-            // can't do property wrappers on override declarations
-            return """
-            @usableFromInline let \(wrapperName): \(idlType.propertyWrapper(readonly: readonly))<\(idlType)>
-            @inlinable override public var \(name): \(idlType) {
-                get { \(wrapperName).wrappedValue }
-                \(readonly ? "" : "set { \(wrapperName).wrappedValue = newValue }")
-            }
-            """
-        } else if ModuleState.constructor == nil || ModuleState.static {
-            // can't do property wrappers on extensions
-            let propertyWrapper = idlType.propertyWrapper(readonly: readonly)
-            if [SwiftSource.readOnlyAttribute, .readWriteAttribute].contains(propertyWrapper) {
-                let setter: SwiftSource = """
-                nonmutating set { jsObject[\(ModuleState.source(for: name))] = _toJSValue(newValue) }
-                """
 
-                return """
-                @inlinable public\(raw: ModuleState.static ? " static" : "") var \(name): \(idlType) {
-                    get { jsObject[\(ModuleState.source(for: name))]\(idlType.fromJSValue) }
-                    \(readonly ? "" : setter)
-                }
-                """
+            return ""
+        } else {
+            let stringKey = ModuleState.source(for: name)
+            let getter: SwiftSource
+            let setter: SwiftSource
+
+            if let closure = idlType.closurePattern {
+                getter = "get { \(closure.getter(name: stringKey)) }"
+                setter = "set { \(closure.setter(name: stringKey)) }"
             } else {
-                let setter: SwiftSource = """
-                nonmutating set { \(
-                    idlType.propertyWrapper(readonly: readonly))[\(ModuleState.source(for: name)
-                ), in: jsObject] = newValue }
-                """
+                getter = "get { jsObject[\(stringKey)]\(idlType.fromJSValue) }"
+                setter = "set { jsObject[\(stringKey)] = _toJSValue(newValue) }"
+            }
 
-                return """
+            return """
                 @inlinable public\(raw: ModuleState.static ? " static" : "") var \(name): \(idlType) {
-                    get { \(idlType.propertyWrapper(readonly: readonly))[\(ModuleState.source(for: name)), in: jsObject] }
+                    \(getter)
                     \(readonly ? "" : setter)
                 }
                 """
-            }
-        } else {
-            return """
-            @\(idlType.propertyWrapper(readonly: readonly))
-            public var \(name): \(idlType)
-            """
         }
     }