Skip to content

Commit 1fdbc94

Browse files
committed
Get rid of c-style for loops
1 parent 176e8c5 commit 1fdbc94

File tree

3 files changed

+11
-8
lines changed

3 files changed

+11
-8
lines changed

Foundation/NSHost.swift

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -84,9 +84,11 @@ public class NSHost : NSObject {
8484
if r != 0 {
8585
return
8686
}
87-
for var res: UnsafeMutablePointer<addrinfo> = res0; res != nil; res = res.memory.ai_next {
87+
var res: UnsafeMutablePointer<addrinfo> = res0
88+
while res != nil {
8889
let family = res.memory.ai_family
8990
if family != AF_INET && family != AF_INET6 {
91+
res = res.memory.ai_next
9092
continue
9193
}
9294
let sa_len: socklen_t = socklen_t((family == AF_INET6) ? sizeof(sockaddr_in6) : sizeof(sockaddr_in))
@@ -101,6 +103,7 @@ public class NSHost : NSObject {
101103
lookupInfo(&_addresses, NI_NUMERICHOST)
102104
lookupInfo(&_names, NI_NAMEREQD)
103105
lookupInfo(&_names, NI_NOFQDN|NI_NAMEREQD)
106+
res = res.memory.ai_next
104107
}
105108

106109
freeaddrinfo(res0)

Foundation/NSXMLParser.swift

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -241,7 +241,7 @@ internal func _NSXMLParserStartElementNs(ctx: _CFXMLInterface, localname: Unsafe
241241
var nsDict = [String:String]()
242242
var attrDict = [String:String]()
243243
if nb_attributes + nb_namespaces > 0 {
244-
for var idx = 0; idx < Int(nb_namespaces) * 2; idx += 2 {
244+
for idx in 0.stride(to: Int(nb_namespaces) * 2, by: 2) {
245245
var namespaceNameString: String?
246246
var asAttrNamespaceNameString: String?
247247
if namespaces[idx] != nil {
@@ -271,7 +271,7 @@ internal func _NSXMLParserStartElementNs(ctx: _CFXMLInterface, localname: Unsafe
271271
parser._pushNamespaces(nsDict)
272272
}
273273

274-
for var idx = 0; idx < Int(nb_attributes) * 5; idx += 5 {
274+
for idx in 0.stride(to: Int(nb_attributes) * 5, by: 5) {
275275
if attributes[idx] == nil {
276276
continue
277277
}

TestFoundation/TestNSCharacterSet.swift

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -63,29 +63,29 @@ class TestNSCharacterSet : XCTestCase {
6363

6464
func test_Range() {
6565
let cset1 = NSCharacterSet(range: NSMakeRange(0x20, 40))
66-
for var idx: unichar = 0; idx < 0xFFFF; idx++ {
66+
for idx: unichar in 0..<0xFFFF {
6767
XCTAssertEqual(cset1.characterIsMember(idx), (idx >= 0x20 && idx < 0x20 + 40 ? true : false))
6868
}
6969

7070
let cset2 = NSCharacterSet(range: NSMakeRange(0x0000, 0xFFFF))
71-
for var idx: unichar = 0; idx < 0xFFFF; idx++ {
71+
for idx: unichar in 0..<0xFFFF {
7272
XCTAssertEqual(cset2.characterIsMember(idx), true)
7373
}
7474

7575
let cset3 = NSCharacterSet(range: NSMakeRange(0x0000, 10))
76-
for var idx: unichar = 0; idx < 0xFFFF; idx++ {
76+
for idx: unichar in 0..<0xFFFF {
7777
XCTAssertEqual(cset3.characterIsMember(idx), (idx < 10 ? true : false))
7878
}
7979

8080
let cset4 = NSCharacterSet(range: NSMakeRange(0x20, 0))
81-
for var idx: unichar = 0; idx < 0xFFFF; idx++ {
81+
for idx: unichar in 0..<0xFFFF {
8282
XCTAssertEqual(cset4.characterIsMember(idx), false)
8383
}
8484
}
8585

8686
func test_String() {
8787
let cset = NSCharacterSet(charactersInString: "abcABC")
88-
for var idx: unichar = 0; idx < 0xFFFF; idx++ {
88+
for idx: unichar in 0..<0xFFFF {
8989
XCTAssertEqual(cset.characterIsMember(idx), (idx >= unichar(unicodeScalarLiteral: "a") && idx <= unichar(unicodeScalarLiteral: "c")) || (idx >= unichar(unicodeScalarLiteral: "A") && idx <= unichar(unicodeScalarLiteral: "C")) ? true : false)
9090
}
9191
}

0 commit comments

Comments
 (0)