[stdlib][Qol] SR-11295 Removing stdlib unnecessary coercions (#27165)

* Removing unnecessary casts from stdlib

* Minor adjustments

* [stdlib][qol] Clean up error variable assign NSError

* Removing unnecessary coercions after removing DeclRefExpr restriction.

Author: LucianoPAlmeida

Darwin/Foundation-swiftoverlay/DateComponents.swift

@@ -204,11 +204,7 @@ public struct DateComponents : ReferenceConvertible, Hashable, Equatable, _Mutab
 
     /// Returns a `Date` calculated from the current components using the `calendar` property.
     public var date: Date? {
-        if let d = _handle.map({$0.date}) {
-            return d as Date
-        } else {
-            return nil
-        }
+        return _handle.map { $0.date } 
     }
 
     // MARK: - Generic Setter/Getters

Darwin/Foundation-swiftoverlay/FileManager.swift

@@ -48,7 +48,7 @@ extension FileManager {
         return __NSFileManagerEnumeratorAtURL(self, url, keys, mask, { (url, error) in
             var errorResult = true
             if let h = handler {
-                errorResult = h(url as URL, error)
+                errorResult = h(url, error)
             }
             return errorResult
         })

Darwin/Foundation-swiftoverlay/IndexSet.swift

@@ -600,7 +600,7 @@ public struct IndexSet : ReferenceConvertible, Equatable, BidirectionalCollectio
                     stop.pointee = true
                     return false
                 }
-            }) as IndexSet
+            })
             if let e = error {
                 throw e
             } else {

Darwin/Foundation-swiftoverlay/NSCoder.swift

@@ -53,7 +53,7 @@ extension NSCoder {
     if let theClasses = classes {
       classesAsNSObjects = NSSet(array: theClasses.map { $0 as AnyObject })
     }
-    return __NSCoderDecodeObjectOfClassesForKey(self, classesAsNSObjects, key, nil).map { $0 as Any }
+    return __NSCoderDecodeObjectOfClassesForKey(self, classesAsNSObjects, key, nil).map { $0 }
   }
 
   @nonobjc
@@ -62,7 +62,7 @@ extension NSCoder {
     var error: NSError?
     let result = __NSCoderDecodeObject(self, &error)
     try resolveError(error)
-    return result.map { $0 as Any }
+    return result.map { $0 }
   }
 
   @available(*, unavailable, renamed: "decodeTopLevelObject(forKey:)")
@@ -125,7 +125,7 @@ extension NSCoder {
     }
     let result = __NSCoderDecodeObjectOfClassesForKey(self, classesAsNSObjects, key, &error)
     try resolveError(error)
-    return result.map { $0 as Any }
+    return result.map { $0 }
   }
 }
 

Darwin/Foundation-swiftoverlay/NSDictionary.swift

@@ -64,7 +64,7 @@ extension Dictionary {
 extension Dictionary : _ObjectiveCBridgeable {
   @_semantics("convertToObjectiveC")
   public func _bridgeToObjectiveC() -> NSDictionary {
-    return unsafeBitCast(_bridgeToObjectiveCImpl() as AnyObject,
+    return unsafeBitCast(_bridgeToObjectiveCImpl(),
                          to: NSDictionary.self)
   }
 

Darwin/Foundation-swiftoverlay/NSError.swift

@@ -234,8 +234,7 @@ public func _getErrorDefaultUserInfo<T: Error>(_ error: T)
     if domain != NSCocoaErrorDomain {
       _errorDomainUserInfoProviderQueue.sync {
         if NSError.userInfoValueProvider(forDomain: domain) != nil { return }
-        NSError.setUserInfoValueProvider(forDomain: domain) { (nsError, key) in
-          let error = nsError as Error
+        NSError.setUserInfoValueProvider(forDomain: domain) { (error, key) in
 
           switch key {
           case NSLocalizedDescriptionKey:

Darwin/Foundation-swiftoverlay/NSSet.swift

@@ -55,7 +55,7 @@ extension NSOrderedSet : Sequence {
 extension Set : _ObjectiveCBridgeable {
   @_semantics("convertToObjectiveC")
   public func _bridgeToObjectiveC() -> NSSet {
-    return unsafeBitCast(_bridgeToObjectiveCImpl() as AnyObject, to: NSSet.self)
+    return unsafeBitCast(_bridgeToObjectiveCImpl(), to: NSSet.self)
   }
 
   public static func _forceBridgeFromObjectiveC(_ s: NSSet, result: inout Set?) {

Darwin/Foundation-swiftoverlay/NSStringAPI.swift

@@ -353,7 +353,7 @@ extension String {
     usedEncoding: inout Encoding
   ) throws {
     var enc: UInt = 0
-    let ns = try NSString(contentsOf: url as URL, usedEncoding: &enc)
+    let ns = try NSString(contentsOf: url, usedEncoding: &enc)
     usedEncoding = Encoding(rawValue: enc)
     self = String._unconditionallyBridgeFromObjectiveC(ns)
   }
@@ -566,7 +566,7 @@ extension StringProtocol where Index == String.Index {
   /// Case transformations aren’t guaranteed to be symmetrical or to produce
   /// strings of the same lengths as the originals.
   public var capitalized: String {
-    return _ns.capitalized as String
+    return _ns.capitalized
   }
 
   // @property (readonly, copy) NSString *localizedCapitalizedString NS_AVAILABLE(10_11, 9_0);
@@ -583,7 +583,7 @@ extension StringProtocol where Index == String.Index {
   /// Returns a capitalized representation of the string
   /// using the specified locale.
   public func capitalized(with locale: Locale?) -> String {
-    return _ns.capitalized(with: locale) as String
+    return _ns.capitalized(with: locale)
   }
 
   // - (NSComparisonResult)caseInsensitiveCompare:(NSString *)aString
@@ -1166,7 +1166,7 @@ extension StringProtocol where Index == String.Index {
   /// locale.
   @available(macOS 10.11, iOS 9.0, *)
   public var localizedUppercase: String {
-    return _ns.localizedUppercase as String
+    return _ns.localizedUppercase
   }
 
   // - (NSString *)uppercaseStringWithLocale:(Locale *)locale

Darwin/Foundation-swiftoverlay/Progress.swift

@@ -17,7 +17,7 @@ extension Progress {
     public var estimatedTimeRemaining: TimeInterval? {
         get {
             guard let v = self.__estimatedTimeRemaining else { return nil }
-            return v.doubleValue as TimeInterval
+            return v.doubleValue
         }
         set {
             guard let nv = newValue else {

Darwin/Foundation-swiftoverlay/String.swift

@@ -32,7 +32,7 @@ extension String : _ObjectiveCBridgeable {
     // This method should not do anything extra except calling into the
     // implementation inside core.  (These two entry points should be
     // equivalent.)
-    return unsafeBitCast(_bridgeToObjectiveCImpl() as AnyObject, to: NSString.self)
+    return unsafeBitCast(_bridgeToObjectiveCImpl(), to: NSString.self)
   }
 
   public static func _forceBridgeFromObjectiveC(

Darwin/Foundation-swiftoverlay/URL.swift

@@ -1090,7 +1090,7 @@ public struct URL : ReferenceConvertible, Equatable {
     /// Returns bookmark data for the URL, created with specified options and resource values.
     public func bookmarkData(options: BookmarkCreationOptions = [], includingResourceValuesForKeys keys: Set<URLResourceKey>? = nil, relativeTo url: URL? = nil) throws -> Data {
         let result = try _url.bookmarkData(options: options, includingResourceValuesForKeys: keys.flatMap { Array($0) }, relativeTo: url)
-        return result as Data
+        return result
     }
 
     /// Returns the resource values for properties identified by a specified array of keys contained in specified bookmark data. If the result dictionary does not contain a resource value for one or more of the requested resource keys, it means those resource properties are not available in the bookmark data.
@@ -1107,7 +1107,7 @@ public struct URL : ReferenceConvertible, Equatable {
     /// Initializes and returns bookmark data derived from an alias file pointed to by a specified URL. If bookmarkFileURL refers to an alias file created prior to OS X v10.6 that contains Alias Manager information but no bookmark data, this method synthesizes bookmark data for the file.
     public static func bookmarkData(withContentsOf url: URL) throws -> Data {
         let result = try NSURL.bookmarkData(withContentsOf: url)
-        return result as Data
+        return result
     }
 
     /// Given an NSURL created by resolving a bookmark data created with security scope, make the resource referenced by the url accessible to the process. When access to this resource is no longer needed the client must call stopAccessingSecurityScopedResource. Each call to startAccessingSecurityScopedResource must be balanced with a call to stopAccessingSecurityScopedResource (Note: this is not reference counted).