Fix bug [SR-4391]: index check should be || instead of &&

rxwei · rxwei · commit 9030208a4d13 · 2017-03-29T02:05:41.000-05:00
diff --git a/Foundation/NSOrderedSet.swift b/Foundation/NSOrderedSet.swift
@@ -148,7 +148,7 @@ extension NSOrderedSet {
     open func objects(at indexes: IndexSet) -> [Any] {
         var entries = [Any]()
         for idx in indexes {
-            if idx >= count && idx < 0 {
+            if idx >= count || idx < 0 {
                 fatalError("\(self): Index out of bounds")
             }
             entries.append(object(at: idx))

Original file line number	Diff line number	Diff line change
`@@ -148,7 +148,7 @@ extension NSOrderedSet {`
`148`	`148`	`open func objects(at indexes: IndexSet) -> [Any] {`
`149`	`149`	`var entries = [Any]()`
`150`	`150`	`for idx in indexes {`
`151`		`- if idx >= count && idx < 0 {`
	`151`	`+ if idx >= count \|\| idx < 0 {`
`152`	`152`	`fatalError("\(self): Index out of bounds")`
`153`	`153`	`}`
`154`	`154`	`entries.append(object(at: idx))`