[stdlib] tweaks in associated object code

Author: glessard

stdlib/public/core/StringGuts.swift

@@ -430,14 +430,15 @@ extension _StringGuts {
   }
 
   internal func getAssociatedStorage() -> __StringStorage? {
+    _internalInvariant(_object.hasObjCBridgeableObject)
     let getter = unsafe unsafeBitCast(
       getGetAssociatedObjectPtr(),
       to: (@convention(c)(
         AnyObject,
         UnsafeRawPointer
       ) -> UnsafeRawPointer?).self
     )
-    _precondition(_object.hasObjCBridgeableObject)
+
     if let assocPtr = unsafe getter(
       _object.objCBridgeableObject,
       Self.associationKey
@@ -450,6 +451,7 @@ extension _StringGuts {
   }
 
   internal func setAssociatedStorage(_ storage: __StringStorage) {
+    _internalInvariant(_object.hasObjCBridgeableObject)
     let setter = unsafe unsafeBitCast(
       getSetAssociatedObjectPtr(),
       to: (@convention(c)(
@@ -469,7 +471,7 @@ extension _StringGuts {
   }
 
   internal func getOrAllocateAssociatedStorage() -> __StringStorage {
-    _precondition(_object.hasObjCBridgeableObject)
+    _internalInvariant(_object.hasObjCBridgeableObject)
     let unwrapped: __StringStorage
     // libobjc already provides the necessary memory barriers for
     // double checked locking to be safe, per comments on
@@ -483,8 +485,10 @@ extension _StringGuts {
         unwrapped = storage
       } else {
         var contents = String.UnicodeScalarView()
-        // always reserve a size larger than a small string
-        contents.reserveCapacity(Swift.max(_SmallString.capacity + 1, 1 + count + count >> 1))
+        // always reserve a capacity larger than a small string
+        contents.reserveCapacity(
+          Swift.max(_SmallString.capacity + 1, count + count >> 1)
+        )
         for c in String.UnicodeScalarView(self) {
           contents.append(c)
         }