From a8e91d37512c20e2412e60083e4c87f43b824a1f Mon Sep 17 00:00:00 2001 From: Greg Bolsinga Date: Wed, 15 May 2019 10:58:13 -0700 Subject: [PATCH] Remove local shadow variable `self.properties` should be used for clarity. (#202) --- Sources/Core/ObjCModelRenderer.swift | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/Sources/Core/ObjCModelRenderer.swift b/Sources/Core/ObjCModelRenderer.swift index 6bb8cf75..553ff721 100644 --- a/Sources/Core/ObjCModelRenderer.swift +++ b/Sources/Core/ObjCModelRenderer.swift @@ -120,8 +120,6 @@ public struct ObjCModelRenderer: ObjCFileRenderer { } func renderRoots() -> [ObjCIR.Root] { - let properties: [(Parameter, SchemaObjectProperty)] = rootSchema.properties.map { $0 } // Convert [String:Schema] -> [(String, Schema)] - let protocols: [String: [ObjCIR.Method]] = [ "NSSecureCoding": [self.renderSupportsSecureCoding(), self.renderInitWithCoder(), self.renderEncodeWithCoder()], "NSCopying": [ObjCIR.method("- (id)copyWithZone:(NSZone *)zone") { ["return self;"] }], @@ -143,13 +141,13 @@ public struct ObjCModelRenderer: ObjCFileRenderer { } } - let enumRoots = self.properties.flatMap { (param, prop) -> [ObjCIR.Root] in + let enumRoots = properties.flatMap { (param, prop) -> [ObjCIR.Root] in enumRoot(from: prop.schema, param: param) } // TODO: Synthesize oneOf ADT Classes and Class Extension // TODO: (rmalik): Clean this up, too much copy / paste here to support oneOf cases - let adtRoots = self.properties.flatMap { (param, prop) -> [ObjCIR.Root] in + let adtRoots = properties.flatMap { (param, prop) -> [ObjCIR.Root] in switch prop.schema { case let .oneOf(types: possibleTypes): let objProps = possibleTypes.map { SchemaObjectProperty(schema: $0, nullability: $0.isObjCPrimitiveType ? nil : .nullable) }