Skip to content

Commit 601d840

Browse files
jrose-applejrose-apple
committed
Update for SE-0055: Making pointer nullability explicit.
See swiftlang/swift#1878.
1 parent 5eebcc8 commit 601d840

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

55 files changed

+431
-534
lines changed

‎CoreFoundation/Base.subproj/ForFoundationOnly.h‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -588,7 +588,7 @@ enum {
588588
// This is for NSNumberFormatter use only!
589589
CF_EXPORT void *_CFNumberFormatterGetFormatter(CFNumberFormatterRef formatter);
590590

591-
CF_EXPORT void _CFDataInit(CFMutableDataRef memory, CFOptionFlags flags, CFIndex capacity, const uint8_t *bytes, CFIndex length, Boolean noCopy);
591+
CF_EXPORT void _CFDataInit(CFMutableDataRef memory, CFOptionFlags flags, CFIndex capacity, const uint8_t * _Nullable bytes, CFIndex length, Boolean noCopy);
592592
CF_EXPORT CFRange _CFDataFindBytes(CFDataRef data, CFDataRef dataToFind, CFRange searchRange, CFDataSearchFlags compareOptions);
593593

594594

‎CoreFoundation/Base.subproj/ForSwiftFoundationOnly.h‎

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,7 @@ struct _NSObjectBridge {
5454
struct _NSArrayBridge {
5555
CFIndex (*_Nonnull count)(CFTypeRef obj);
5656
_Nonnull CFTypeRef (*_Nonnull objectAtIndex)(CFTypeRef obj, CFIndex index);
57-
void (*_Nonnull getObjects)(CFTypeRef array, CFRange range, CFTypeRef _Nonnull * _Nonnull values);
57+
void (*_Nonnull getObjects)(CFTypeRef array, CFRange range, CFTypeRef _Nullable *_Nonnull values);
5858
};
5959

6060
struct _NSMutableArrayBridge {
@@ -65,19 +65,19 @@ struct _NSMutableArrayBridge {
6565
void (*exchangeObjectAtIndex)(CFTypeRef array, CFIndex idx1, CFIndex idx2);
6666
void (*removeObjectAtIndex)(CFTypeRef array, CFIndex idx);
6767
void (*removeAllObjects)(CFTypeRef array);
68-
void (*replaceObjectsInRange)(CFTypeRef array, CFRange range, CFTypeRef _Nonnull * _Nonnull newValues, CFIndex newCount);
68+
void (*replaceObjectsInRange)(CFTypeRef array, CFRange range, CFTypeRef _Nonnull *_Nullable newValues, CFIndex newCount);
6969
};
7070

7171
struct _NSDictionaryBridge {
7272
CFIndex (*count)(CFTypeRef dictionary);
7373
CFIndex (*countForKey)(CFTypeRef dictionary, CFTypeRef key);
7474
bool (*containsKey)(CFTypeRef dictionary, CFTypeRef key);
7575
_Nullable CFTypeRef (*_Nonnull objectForKey)(CFTypeRef dictionary, CFTypeRef key);
76-
bool (*_getValueIfPresent)(CFTypeRef dictionary, CFTypeRef key, CFTypeRef _Nonnull * _Nullable value);
76+
bool (*_getValueIfPresent)(CFTypeRef dictionary, CFTypeRef key, CFTypeRef _Nullable *_Nonnull value);
7777
CFIndex (*__getValue)(CFTypeRef dictionary, CFTypeRef value, CFTypeRef key);
7878
bool (*containsObject)(CFTypeRef dictionary, CFTypeRef value);
7979
CFIndex (*countForObject)(CFTypeRef dictionary, CFTypeRef value);
80-
void (*getObjects)(CFTypeRef dictionary, CFTypeRef _Nonnull * _Nonnull valuebuf, CFTypeRef _Nonnull * _Nonnull keybuf);
80+
void (*getObjects)(CFTypeRef dictionary, CFTypeRef _Nullable *_Nullable valuebuf, CFTypeRef _Nullable *_Nullable keybuf);
8181
void (*__apply)(CFTypeRef dictionary, void (*applier)(CFTypeRef key, CFTypeRef value, void *context), void *context);
8282
};
8383

@@ -104,7 +104,7 @@ struct _NSStringBridge {
104104
CFIndex (*length)(CFTypeRef str);
105105
UniChar (*characterAtIndex)(CFTypeRef str, CFIndex idx);
106106
void (*getCharacters)(CFTypeRef str, CFRange range, UniChar *buffer);
107-
CFIndex (*__getBytes)(CFTypeRef str, CFStringEncoding encoding, CFRange range, uint8_t *buffer, CFIndex maxBufLen, CFIndex *usedBufLen);
107+
CFIndex (*__getBytes)(CFTypeRef str, CFStringEncoding encoding, CFRange range, uint8_t *_Nullable buffer, CFIndex maxBufLen, CFIndex *_Nullable usedBufLen);
108108
const char *_Nullable (*_Nonnull _fastCStringContents)(CFTypeRef str);
109109
const UniChar *_Nullable (*_Nonnull _fastCharacterContents)(CFTypeRef str);
110110
bool (*_getCString)(CFTypeRef str, char *buffer, size_t len, UInt32 encoding);
@@ -123,15 +123,15 @@ struct _NSMutableStringBridge {
123123

124124
struct _NSXMLParserBridge {
125125
_CFXMLInterface _Nullable (*_Nonnull currentParser)();
126-
_CFXMLInterfaceParserInput _Nonnull (*_Nonnull _xmlExternalEntityWithURL)(_CFXMLInterface interface, const char *url, const char * identifier, _CFXMLInterfaceParserContext context, _CFXMLInterfaceExternalEntityLoader originalLoaderFunction);
126+
_CFXMLInterfaceParserInput _Nullable (*_Nonnull _xmlExternalEntityWithURL)(_CFXMLInterface interface, const char *url, const char * identifier, _CFXMLInterfaceParserContext context, _CFXMLInterfaceExternalEntityLoader originalLoaderFunction);
127127

128128
_CFXMLInterfaceParserContext _Nonnull (*_Nonnull getContext)(_CFXMLInterface ctx);
129129

130130
void (*internalSubset)(_CFXMLInterface ctx, const unsigned char *name, const unsigned char *ExternalID, const unsigned char *SystemID);
131131
int (*isStandalone)(_CFXMLInterface ctx);
132132
int (*hasInternalSubset)(_CFXMLInterface ctx);
133133
int (*hasExternalSubset)(_CFXMLInterface ctx);
134-
_CFXMLInterfaceEntity _Nonnull (*_Nonnull getEntity)(_CFXMLInterface ctx, const unsigned char *name);
134+
_CFXMLInterfaceEntity _Nullable (*_Nonnull getEntity)(_CFXMLInterface ctx, const unsigned char *name);
135135
void (*notationDecl)(_CFXMLInterface ctx,
136136
const unsigned char *name,
137137
const unsigned char *publicId,
@@ -156,16 +156,16 @@ struct _NSXMLParserBridge {
156156
void (*endDocument)(_CFXMLInterface ctx);
157157
void (*startElementNs)(_CFXMLInterface ctx,
158158
const unsigned char *localname,
159-
const unsigned char *prefix,
159+
const unsigned char *_Nullable prefix,
160160
const unsigned char *URI,
161161
int nb_namespaces,
162-
const unsigned char *_Nonnull *_Nonnull namespaces,
162+
const unsigned char *_Nullable *_Nonnull namespaces,
163163
int nb_attributes,
164164
int nb_defaulted,
165-
const unsigned char *_Nonnull *_Nonnull attributes);
165+
const unsigned char *_Nullable *_Nonnull attributes);
166166
void (*endElementNs)(_CFXMLInterface ctx,
167167
const unsigned char *localname,
168-
const unsigned char *prefix,
168+
const unsigned char *_Nullable prefix,
169169
const unsigned char *URI);
170170
void (*characters)(_CFXMLInterface ctx,
171171
const unsigned char *ch,
@@ -239,7 +239,7 @@ extern CFWriteStreamRef _CFWriteStreamCreateFromFileDescriptor(CFAllocatorRef al
239239
extern _Nullable CFDateRef CFCalendarCopyGregorianStartDate(CFCalendarRef calendar);
240240
extern void CFCalendarSetGregorianStartDate(CFCalendarRef calendar, CFDateRef date);
241241

242-
CF_PRIVATE CF_EXPORT char *_Nonnull*_Nonnull _CFEnviron(void);
242+
CF_PRIVATE CF_EXPORT char *_Nullable *_Nonnull _CFEnviron(void);
243243

244244
CF_EXPORT void CFLog1(CFLogLevel lev, CFStringRef message);
245245

‎CoreFoundation/Collections.subproj/CFData.c‎

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -361,7 +361,10 @@ CF_SWIFT_EXPORT void _CFDataInit(CFMutableDataRef memory, CFOptionFlags flags, C
361361
}
362362

363363
__CFDataSetNeedsToZero(memory, !cleared);
364-
CFDataReplaceBytes(memory, CFRangeMake(0, 0), bytes, length);
364+
// Avoid copying from null pointers.
365+
if (length > 0) {
366+
CFDataReplaceBytes(memory, CFRangeMake(0, 0), bytes, length);
367+
}
365368
}
366369
__CFSetMutableVariety(memory, __CFMutableVarietyFromFlags(flags));
367370
}

‎CoreFoundation/Parsing.subproj/CFXMLInterface.h‎

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -136,7 +136,7 @@ typedef void* _CFXMLEntityPtr;
136136
typedef void* _CFXMLDTDPtr;
137137
typedef void* _CFXMLDTDNodePtr;
138138

139-
_CFXMLNodePtr _CFXMLNewNode(_CFXMLNamespacePtr namespace, const char* name);
139+
_CFXMLNodePtr _CFXMLNewNode(_CFXMLNamespacePtr _Nullable namespace, const char* name);
140140
_CFXMLNodePtr _CFXMLCopyNode(_CFXMLNodePtr node, bool recursive);
141141

142142
_CFXMLDocPtr _CFXMLNewDoc(const unsigned char* version);
@@ -147,7 +147,7 @@ _CFXMLNodePtr _CFXMLNewProperty(_CFXMLNodePtr _Nullable node, const unsigned cha
147147
_CFXMLNamespacePtr _CFXMLNewNamespace(_CFXMLNodePtr _Nullable node, const unsigned char* uri, const unsigned char* prefix);
148148

149149
CF_RETURNS_RETAINED CFStringRef _Nullable _CFXMLNodeURI(_CFXMLNodePtr node);
150-
void _CFXMLNodeSetURI(_CFXMLNodePtr node, const unsigned char* URI);
150+
void _CFXMLNodeSetURI(_CFXMLNodePtr node, const unsigned char* _Nullable URI);
151151

152152
void _CFXMLNodeSetPrivateData(_CFXMLNodePtr node, void* data);
153153
void* _Nullable _CFXMLNodeGetPrivateData(_CFXMLNodePtr node);
@@ -156,7 +156,7 @@ CFIndex _CFXMLNodeGetType(_CFXMLNodePtr node);
156156
const char* _CFXMLNodeGetName(_CFXMLNodePtr node);
157157
void _CFXMLNodeSetName(_CFXMLNodePtr node, const char* name);
158158
CF_RETURNS_RETAINED CFStringRef _Nullable _CFXMLNodeGetContent(_CFXMLNodePtr node);
159-
void _CFXMLNodeSetContent(_CFXMLNodePtr node, const unsigned char* _Nullable content);
159+
void _CFXMLNodeSetContent(_CFXMLNodePtr node, const unsigned char* _Nullable content);
160160
void _CFXMLUnlinkNode(_CFXMLNodePtr node);
161161

162162
_CFXMLNodePtr _Nullable _CFXMLNodeGetFirstChild(_CFXMLNodePtr node);
@@ -179,13 +179,13 @@ void _CFXMLDocSetRootElement(_CFXMLDocPtr doc, _CFXMLNodePtr node);
179179
CF_RETURNS_RETAINED CFStringRef _Nullable _CFXMLDocCharacterEncoding(_CFXMLDocPtr doc);
180180
void _CFXMLDocSetCharacterEncoding(_CFXMLDocPtr doc, const unsigned char* _Nullable encoding);
181181
CF_RETURNS_RETAINED CFStringRef _Nullable _CFXMLDocVersion(_CFXMLDocPtr doc);
182-
void _CFXMLDocSetVersion(_CFXMLDocPtr doc, const unsigned char* version);
182+
void _CFXMLDocSetVersion(_CFXMLDocPtr doc, const unsigned char* _Nullable version);
183183
int _CFXMLDocProperties(_CFXMLDocPtr doc);
184184
void _CFXMLDocSetProperties(_CFXMLDocPtr doc, int newProperties);
185185
_CFXMLDTDPtr _Nullable _CFXMLDocDTD(_CFXMLDocPtr doc);
186186
void _CFXMLDocSetDTD(_CFXMLDocPtr doc, _CFXMLDTDPtr _Nullable dtd);
187187

188-
CF_RETURNS_RETAINED CFStringRef _Nullable _CFXMLEncodeEntities(_CFXMLDocPtr doc, const unsigned char* string);
188+
CF_RETURNS_RETAINED CFStringRef _Nullable _CFXMLEncodeEntities(_CFXMLDocPtr _Nullable doc, const unsigned char* string);
189189
_CFXMLEntityPtr _Nullable _CFXMLGetDocEntity(_CFXMLDocPtr doc, const char* entity);
190190
_CFXMLEntityPtr _Nullable _CFXMLGetDTDEntity(_CFXMLDocPtr doc, const char* entity);
191191
_CFXMLEntityPtr _Nullable _CFXMLGetParameterEntity(_CFXMLDocPtr doc, const char* entity);
@@ -205,32 +205,32 @@ CF_RETURNS_RETAINED CFStringRef _Nullable _CFXMLNodePrefix(_CFXMLNodePtr node);
205205

206206
bool _CFXMLDocValidate(_CFXMLDocPtr doc, CFErrorRef _Nullable * error);
207207

208-
_CFXMLDTDPtr _CFXMLNewDTD(_CFXMLDocPtr doc, const unsigned char* name, const unsigned char* publicID, const unsigned char* systemID);
208+
_CFXMLDTDPtr _CFXMLNewDTD(_CFXMLDocPtr _Nullable doc, const unsigned char* name, const unsigned char* publicID, const unsigned char* systemID);
209209
_CFXMLDTDNodePtr _Nullable _CFXMLParseDTDNode(const unsigned char* xmlString);
210210
_CFXMLDTDPtr _Nullable _CFXMLParseDTD(const unsigned char* URL);
211211
_CFXMLDTDPtr _Nullable _CFXMLParseDTDFromData(CFDataRef data, CFErrorRef _Nullable * error);
212212
CF_RETURNS_RETAINED CFStringRef _Nullable _CFXMLDTDExternalID(_CFXMLDTDPtr dtd);
213-
void _CFXMLDTDSetExternalID(_CFXMLDTDPtr dtd, const unsigned char* externalID);
213+
void _CFXMLDTDSetExternalID(_CFXMLDTDPtr dtd, const unsigned char* _Nullable externalID);
214214
CF_RETURNS_RETAINED CFStringRef _Nullable _CFXMLDTDSystemID(_CFXMLDTDPtr dtd);
215-
void _CFXMLDTDSetSystemID(_CFXMLDTDPtr dtd, const unsigned char* systemID);
215+
void _CFXMLDTDSetSystemID(_CFXMLDTDPtr dtd, const unsigned char* _Nullable systemID);
216216

217217
_CFXMLDTDNodePtr _Nullable _CFXMLDTDGetElementDesc(_CFXMLDTDPtr dtd, const unsigned char* name);
218218
_CFXMLDTDNodePtr _Nullable _CFXMLDTDGetAttributeDesc(_CFXMLDTDPtr dtd, const unsigned char* elementName, const unsigned char* name);
219219
_CFXMLDTDNodePtr _Nullable _CFXMLDTDGetNotationDesc(_CFXMLDTDPtr dtd, const unsigned char* name);
220220
_CFXMLDTDNodePtr _Nullable _CFXMLDTDGetEntityDesc(_CFXMLDTDPtr dtd, const unsigned char* name);
221221
_CFXMLDTDNodePtr _Nullable _CFXMLDTDGetPredefinedEntity(const unsigned char* name);
222222

223-
_CFXMLDTDNodePtr _Nullable _CFXMLDTDNewElementDesc(_CFXMLDTDPtr dtd, const unsigned char* name);
223+
_CFXMLDTDNodePtr _Nullable _CFXMLDTDNewElementDesc(_CFXMLDTDPtr _Nullable dtd, const unsigned char* _Nullable name);
224224
_CFXMLDTDNodePtr _Nullable _CFXMLDTDNewAttributeDesc(_CFXMLDTDPtr dtd, const unsigned char* name);
225225

226226
CFIndex _CFXMLDTDElementNodeGetType(_CFXMLDTDNodePtr node);
227227
CFIndex _CFXMLDTDEntityNodeGetType(_CFXMLDTDNodePtr node);
228228
CFIndex _CFXMLDTDAttributeNodeGetType(_CFXMLDTDNodePtr node);
229229

230230
CF_RETURNS_RETAINED CFStringRef _Nullable _CFXMLDTDNodeGetSystemID(_CFXMLDTDNodePtr node);
231-
void _CFXMLDTDNodeSetSystemID(_CFXMLDTDNodePtr node, const unsigned char* systemID);
231+
void _CFXMLDTDNodeSetSystemID(_CFXMLDTDNodePtr node, const unsigned char* _Nullable systemID);
232232
CF_RETURNS_RETAINED CFStringRef _Nullable _CFXMLDTDNodeGetPublicID(_CFXMLDTDNodePtr node);
233-
void _CFXMLDTDNodeSetPublicID(_CFXMLDTDNodePtr node, const unsigned char* publicID);
233+
void _CFXMLDTDNodeSetPublicID(_CFXMLDTDNodePtr node, const unsigned char* _Nullable publicID);
234234

235235
void _CFXMLFreeNode(_CFXMLNodePtr node);
236236
void _CFXMLFreeDocument(_CFXMLDocPtr doc);

‎Foundation/NSArray.swift‎

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -58,10 +58,10 @@ public class NSArray : NSObject, NSCopying, NSMutableCopying, NSSecureCoding, NS
5858
self.init(objects: nil, count:0)
5959
}
6060

61-
public required init(objects: UnsafePointer<AnyObject?>, count cnt: Int) {
61+
public required init(objects: UnsafePointer<AnyObject?>?, count cnt: Int) {
6262
_storage.reserveCapacity(cnt)
6363
for idx in 0..<cnt {
64-
_storage.append(objects[idx]!)
64+
_storage.append(objects![idx]!)
6565
}
6666
}
6767

@@ -359,13 +359,13 @@ public class NSArray : NSObject, NSCopying, NSMutableCopying, NSSecureCoding, NS
359359
return NSData(bytesNoCopy: unsafeBitCast(buffer, to: UnsafeMutablePointer<Void>.self), length: count * sizeof(Int), freeWhenDone: true)
360360
}
361361

362-
public func sortedArrayUsingFunction(comparator: @convention(c) (AnyObject, AnyObject, UnsafeMutablePointer<Void>) -> Int, context: UnsafeMutablePointer<Void>) -> [AnyObject] {
362+
public func sortedArrayUsingFunction(comparator: @convention(c) (AnyObject, AnyObject, UnsafeMutablePointer<Void>?) -> Int, context: UnsafeMutablePointer<Void>?) -> [AnyObject] {
363363
return sortedArrayWithOptions([]) { lhs, rhs in
364364
return NSComparisonResult(rawValue: comparator(lhs, rhs, context))!
365365
}
366366
}
367367

368-
public func sortedArrayUsingFunction(comparator: @convention(c) (AnyObject, AnyObject, UnsafeMutablePointer<Void>) -> Int, context: UnsafeMutablePointer<Void>, hint: NSData?) -> [AnyObject] {
368+
public func sortedArrayUsingFunction(comparator: @convention(c) (AnyObject, AnyObject, UnsafeMutablePointer<Void>?) -> Int, context: UnsafeMutablePointer<Void>?, hint: NSData?) -> [AnyObject] {
369369
return sortedArrayWithOptions([]) { lhs, rhs in
370370
return NSComparisonResult(rawValue: comparator(lhs, rhs, context))!
371371
}
@@ -660,10 +660,10 @@ public class NSMutableArray : NSArray {
660660
}
661661
}
662662

663-
public required convenience init(objects: UnsafePointer<AnyObject?>, count cnt: Int) {
663+
public required convenience init(objects: UnsafePointer<AnyObject?>?, count cnt: Int) {
664664
self.init(capacity: cnt)
665665
for idx in 0..<cnt {
666-
_storage.append(objects[idx]!)
666+
_storage.append(objects![idx]!)
667667
}
668668
}
669669

@@ -807,7 +807,7 @@ public class NSMutableArray : NSArray {
807807
}
808808
}
809809

810-
public func sortUsingFunction(compare: @convention(c) (AnyObject, AnyObject, UnsafeMutablePointer<Void>) -> Int, context: UnsafeMutablePointer<Void>) {
810+
public func sortUsingFunction(compare: @convention(c) (AnyObject, AnyObject, UnsafeMutablePointer<Void>?) -> Int, context: UnsafeMutablePointer<Void>?) {
811811
self.setArray(self.sortedArrayUsingFunction(compare, context: context))
812812
}
813813

‎Foundation/NSCFArray.swift‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ internal final class _NSCFArray : NSMutableArray {
2020
fatalError()
2121
}
2222

23-
required init(objects: UnsafePointer<AnyObject?>, count cnt: Int) {
23+
required init(objects: UnsafePointer<AnyObject?>?, count cnt: Int) {
2424
fatalError()
2525
}
2626

@@ -89,7 +89,7 @@ internal func _CFSwiftArrayRemoveAllValues(array: AnyObject) {
8989
(array as! NSMutableArray).removeAllObjects()
9090
}
9191

92-
internal func _CFSwiftArrayReplaceValues(array: AnyObject, _ range: CFRange, _ newValues: UnsafeMutablePointer<Unmanaged<AnyObject>?>, _ newCount: CFIndex) {
92+
internal func _CFSwiftArrayReplaceValues(array: AnyObject, _ range: CFRange, _ newValues: UnsafeMutablePointer<Unmanaged<AnyObject>>?, _ newCount: CFIndex) {
9393
NSUnimplemented()
9494
// (array as! NSMutableArray).replaceObjectsInRange(NSMakeRange(range.location, range.length), withObjectsFromArray: newValues.array(newCount))
9595
}

‎Foundation/NSCFDictionary.swift‎

Lines changed: 5 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -64,11 +64,11 @@ internal final class _NSCFDictionary : NSMutableDictionary {
6464
let cf = dict._cfObject
6565
count = CFDictionaryGetCount(cf)
6666

67-
let keys = UnsafeMutablePointer<UnsafePointer<Void>>(allocatingCapacity: count)
67+
let keys = UnsafeMutablePointer<UnsafePointer<Void>?>(allocatingCapacity: count)
6868
CFDictionaryGetKeysAndValues(cf, keys, nil)
6969

7070
for idx in 0..<count {
71-
let key = unsafeBitCast(keys.advanced(by: idx).pointee, to: NSObject.self)
71+
let key = unsafeBitCast(keys.advanced(by: idx).pointee!, to: NSObject.self)
7272
keyArray.append(key)
7373
}
7474
keys.deinitialize()
@@ -140,18 +140,14 @@ internal func _CFSwiftDictionaryContainsValue(dictionary: AnyObject, value: AnyO
140140
NSUnimplemented()
141141
}
142142

143-
internal func _CFSwiftDictionaryGetValuesAndKeys(dictionary: AnyObject, valuebuf: UnsafeMutablePointer<Unmanaged<AnyObject>?>, keybuf: UnsafeMutablePointer<Unmanaged<AnyObject>?>) {
143+
internal func _CFSwiftDictionaryGetValuesAndKeys(dictionary: AnyObject, valuebuf: UnsafeMutablePointer<Unmanaged<AnyObject>?>?, keybuf: UnsafeMutablePointer<Unmanaged<AnyObject>?>?) {
144144
var idx = 0
145145
if valuebuf == nil && keybuf == nil {
146146
return
147147
}
148148
(dictionary as! NSDictionary).enumerateKeysAndObjectsUsingBlock { key, value, _ in
149-
if valuebuf != nil {
150-
valuebuf[idx] = Unmanaged<AnyObject>.passUnretained(value)
151-
}
152-
if keybuf != nil {
153-
keybuf[idx] = Unmanaged<AnyObject>.passUnretained(key)
154-
}
149+
valuebuf?[idx] = Unmanaged<AnyObject>.passUnretained(value)
150+
keybuf?[idx] = Unmanaged<AnyObject>.passUnretained(key)
155151
idx += 1
156152
}
157153
}

0 commit comments

Comments
 (0)