Switch Foundation _sanityChecks to assertions

Author: airspeedswift

stdlib/public/SDK/Foundation/ExtraStringAPIs.swift

@@ -15,7 +15,7 @@ extension String.UTF16View.Index {
   @available(swift, deprecated: 3.2)
   @available(swift, obsoleted: 4.0)
   public init(_ offset: Int) {
-    precondition(offset >= 0, "Negative UTF16 index offset not allowed")
+    assert(offset >= 0, "Negative UTF16 index offset not allowed")
     self.init(encodedOffset: offset)
   }
 

stdlib/public/SDK/Foundation/NSArray.swift

@@ -31,7 +31,7 @@ extension Array : _ObjectiveCBridgeable {
   /// The provided `NSArray` will be copied to ensure that the copy can
   /// not be mutated by other code.
   internal init(_cocoaArray: NSArray) {
-    precondition(_isBridgedVerbatimToObjectiveC(Element.self),
+    assert(_isBridgedVerbatimToObjectiveC(Element.self),
       "Array can be backed by NSArray only when the element type can be bridged verbatim to Objective-C")
     // FIXME: We would like to call CFArrayCreateCopy() to avoid doing an
     // objc_msgSend() for instances of CoreFoundation types.  We can't do that

stdlib/public/SDK/Foundation/NSDictionary.swift

@@ -35,7 +35,7 @@ extension Dictionary {
   /// The provided `NSDictionary` will be copied to ensure that the copy can
   /// not be mutated by other code.
   public init(_cocoaDictionary: _NSDictionary) {
-    precondition(
+    assert(
       _isBridgedVerbatimToObjectiveC(Key.self) &&
       _isBridgedVerbatimToObjectiveC(Value.self),
       "Dictionary can be backed by NSDictionary storage only when both key and value are bridged verbatim to Objective-C")
@@ -228,8 +228,8 @@ extension NSDictionary {
     let alignment = MemoryLayout<AnyObject>.alignment
     let singleSize = stride * numElems
     let totalSize = singleSize * 2
-    precondition(stride == MemoryLayout<NSCopying>.stride)
-    precondition(alignment == MemoryLayout<NSCopying>.alignment)
+    assert(stride == MemoryLayout<NSCopying>.stride)
+    assert(alignment == MemoryLayout<NSCopying>.alignment)
 
     // Allocate a buffer containing both the keys and values.
     let buffer = UnsafeMutableRawPointer.allocate(

stdlib/public/SDK/Foundation/NSSet.swift

@@ -18,7 +18,7 @@ extension Set {
   /// The provided `NSSet` will be copied to ensure that the copy can
   /// not be mutated by other code.
   public init(_cocoaSet: _NSSet) {
-    precondition(_isBridgedVerbatimToObjectiveC(Element.self),
+    assert(_isBridgedVerbatimToObjectiveC(Element.self),
       "Set can be backed by NSSet _variantStorage only when the member type can be bridged verbatim to Objective-C")
     // FIXME: We would like to call CFSetCreateCopy() to avoid doing an
     // objc_msgSend() for instances of CoreFoundation types.  We can't do that

stdlib/public/SDK/Foundation/NSStringAPI.swift

@@ -1637,7 +1637,7 @@ extension StringProtocol where Index == String.Index {
   public func contains<T : StringProtocol>(_ other: T) -> Bool {
     let r = self.range(of: other) != nil
     if #available(macOS 10.10, iOS 8.0, *) {
-      precondition(r == _ns.contains(other._ephemeralString))
+      assert(r == _ns.contains(other._ephemeralString))
     }
     return r
   }
@@ -1660,7 +1660,7 @@ extension StringProtocol where Index == String.Index {
       of: other, options: .caseInsensitive, locale: Locale.current
     ) != nil
     if #available(macOS 10.10, iOS 8.0, *) {
-      precondition(r ==
+      assert(r ==
         _ns.localizedCaseInsensitiveContains(other._ephemeralString))
     }
     return r