Skip to content

Commit debebf7

Browse files
authored
Merge pull request swiftlang#1074 from spevans/pr_range_deprecation
2 parents d54fcb9 + 47f57cc commit debebf7

File tree

5 files changed

+9
-9
lines changed

5 files changed

+9
-9
lines changed

Foundation/Calendar.swift

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -367,7 +367,7 @@ public struct Calendar : Hashable, Equatable, ReferenceConvertible, _MutableBoxi
367367
/// - parameter component: A component to calculate a range for.
368368
/// - returns: The range, or nil if it could not be calculated.
369369
public func minimumRange(of component: Component) -> Range<Int>? {
370-
return _handle.map { $0.minimumRange(of: Calendar._toCalendarUnit([component])).toRange() }
370+
return _handle.map { Range($0.minimumRange(of: Calendar._toCalendarUnit([component]))) }
371371
}
372372

373373
/// The maximum range limits of the values that a given component can take on in the receive
@@ -376,7 +376,7 @@ public struct Calendar : Hashable, Equatable, ReferenceConvertible, _MutableBoxi
376376
/// - parameter component: A component to calculate a range for.
377377
/// - returns: The range, or nil if it could not be calculated.
378378
public func maximumRange(of component: Component) -> Range<Int>? {
379-
return _handle.map { $0.maximumRange(of: Calendar._toCalendarUnit([component])).toRange() }
379+
return _handle.map { Range($0.maximumRange(of: Calendar._toCalendarUnit([component]))) }
380380
}
381381

382382

@@ -393,7 +393,7 @@ public struct Calendar : Hashable, Equatable, ReferenceConvertible, _MutableBoxi
393393
/// - parameter date: The absolute time for which the calculation is performed.
394394
/// - returns: The range of absolute time values smaller can take on in larger at the time specified by date. Returns `nil` if larger is not logically bigger than smaller in the calendar, or the given combination of components does not make sense (or is a computation which is undefined).
395395
public func range(of smaller: Component, in larger: Component, for date: Date) -> Range<Int>? {
396-
return _handle.map { $0.range(of: Calendar._toCalendarUnit([smaller]), in: Calendar._toCalendarUnit([larger]), for: date).toRange() }
396+
return _handle.map { Range($0.range(of: Calendar._toCalendarUnit([smaller]), in: Calendar._toCalendarUnit([larger]), for: date)) }
397397
}
398398

399399
/// Returns, via two inout parameters, the starting time and duration of a given calendar component that contains a given date.

Foundation/NSArray.swift

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -238,7 +238,7 @@ open class NSArray : NSObject, NSCopying, NSMutableCopying, NSSecureCoding, NSCo
238238
objects.reserveCapacity(objects.count + range.length)
239239

240240
if type(of: self) === NSArray.self || type(of: self) === NSMutableArray.self {
241-
objects += _storage[range.toRange()!].map { _SwiftValue.fetch(nonOptional: $0) }
241+
objects += _storage[Range(range)!].map { _SwiftValue.fetch(nonOptional: $0) }
242242
return
243243
}
244244

@@ -495,7 +495,7 @@ open class NSArray : NSObject, NSCopying, NSMutableCopying, NSSecureCoding, NSCo
495495
return []
496496
}
497497

498-
let swiftRange = range.toRange()!
498+
let swiftRange = Range(range)!
499499
return allObjects[swiftRange].sorted { lhs, rhs in
500500
return cmptr(lhs, rhs) == .orderedAscending
501501
}
@@ -784,7 +784,7 @@ open class NSMutableArray : NSArray {
784784

785785
open func removeObjects(in range: NSRange) {
786786
if type(of: self) === NSMutableArray.self {
787-
_storage.removeSubrange(range.toRange()!)
787+
_storage.removeSubrange(Range(range)!)
788788
} else {
789789
for idx in range.toCountableRange()!.reversed() {
790790
removeObject(at: idx)

Foundation/NSData.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -569,7 +569,7 @@ open class NSData : NSObject, NSCopying, NSMutableCopying, NSSecureCoding {
569569
open func range(of dataToFind: Data, options mask: SearchOptions = [], in searchRange: NSRange) -> NSRange {
570570
let dataToFind = dataToFind._nsObject
571571
guard dataToFind.length > 0 else {return NSRange(location: NSNotFound, length: 0)}
572-
guard let searchRange = searchRange.toRange() else {fatalError("invalid range")}
572+
guard let searchRange = Range(searchRange) else {fatalError("invalid range")}
573573

574574
precondition(searchRange.upperBound <= self.length, "range outside the bounds of data")
575575

Foundation/NSOrderedSet.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -553,7 +553,7 @@ extension NSMutableOrderedSet {
553553
NSUnimplemented()
554554
}
555555

556-
let swiftRange = range.toRange()!
556+
let swiftRange = Range(range)!
557557
_orderedStorage[swiftRange].sort { lhs, rhs in
558558
return cmptr(_SwiftValue.fetch(nonOptional: lhs), _SwiftValue.fetch(nonOptional: rhs)) == .orderedAscending
559559
}

TestFoundation/TestNSRange.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,7 @@ class TestNSRange : XCTestCase {
6868
func test_NSRangeBridging() {
6969
let swiftRange: Range<Int> = 1..<7
7070
let range = NSRange(swiftRange)
71-
let swiftRange2 = range.toRange()
71+
let swiftRange2 = Range(range)
7272
XCTAssertEqual(swiftRange, swiftRange2)
7373
}
7474

0 commit comments

Comments
 (0)