Style fixes

Author: stevapple

Sources/SwiftDriver/Driver/OutputFileMap.swift

@@ -138,7 +138,7 @@ public struct OutputFileMap: Hashable, Codable {
     try fileSystem.writeFileContents(file, bytes: ByteString(contents))
   }
 
-  /// Human-readable texual representation
+  /// Human-readable textual representation
   var description: String {
     var result = ""
     func outputPairDescription(inputPath: VirtualPath, outputPair: (FileType, VirtualPath))
@@ -229,7 +229,7 @@ fileprivate struct OutputFileMapJSON: Codable {
 
   /// Converts into virtual path entries.
   func toVirtualOutputFileMap() throws -> [VirtualPath : [FileType : VirtualPath]] {
-    // Add filter here due to a bug of TSC, see swift-tools-support-core#191
+    // TODO: Remove the filter after swift-tools-support-core#191
     Dictionary(try entries.filter { $0.0 != "" }.map { input, entry in
       (try VirtualPath(path: input), try entry.paths.mapValues(VirtualPath.init(path:)))
     }, uniquingKeysWith: { $1 })

Sources/SwiftDriver/IncrementalCompilation/DirectAndTransitiveCollections.swift

@@ -9,6 +9,7 @@
 // See https://swift.org/CONTRIBUTORS.txt for the list of Swift project authors
 //
 //===----------------------------------------------------------------------===//
+
 // Use the type system to ensure that dependencies are transitively closed
 // without doing too much work at the leaves of the call tree
 public struct Transitively {}
@@ -81,4 +82,4 @@ public typealias TransitivelyInvalidatedSourceSet = InvalidatedSet<Transitively,
 public typealias TransitivelyInvalidatedInputArray = InvalidatedArray<Transitively, TypedVirtualPath>
 public typealias TransitivelyInvalidatedInputSet = InvalidatedSet<Transitively, TypedVirtualPath>
 public typealias DirectlyInvalidatedNodeArray = InvalidatedArray<Directly, ModuleDependencyGraph.Node>
-public typealias DirectlyInvalidatedNodeSet = InvalidatedSet<Directly, ModuleDependencyGraph.Node>
+public typealias DirectlyInvalidatedNodeSet = InvalidatedSet<Directly, ModuleDependencyGraph.Node>

Tests/SwiftDriverTests/IncrementalCompilationTests.swift

@@ -91,7 +91,7 @@ final class NonincrementalCompilationTests: XCTestCase {
                         VirtualPath(path: "/Volumes/gazorp.swift"):
                           InputInfo(status: .needsNonCascadingBuild,
                                     previousModTime:  Date(legacyDriverSecsAndNanos: [0, 0]))
-                      ])
+                       ])
   #endif
   }
 

Tests/SwiftDriverTests/XCTestManifests.swift

@@ -117,6 +117,8 @@ extension NonincrementalCompilationTests {
     //   `swift test --generate-linuxmain`
     // to regenerate.
     #if os(Windows)
+    // FIXME: There's some frustrating bug with Yams' date decoding, disable tests on Windows
+    // to prevent unexpected crashes.
     static let __allTests__NonincrementalCompilationTests: [(String, (XCTestCase) -> () -> Void)] = []
     #else
     static let __allTests__NonincrementalCompilationTests = [