Merge pull request #575 from milseman/noescape_by_default

[noescape by default] Add explicit @escaping as needed

Author: milseman

Sources/Commands/SwiftBuildTool.swift

@@ -68,7 +68,7 @@ private enum BuildToolFlag: Argument {
     case ignoreDependencies
     case verbose(Int)
 
-    init?(argument: String, pop: () -> String?) throws {
+    init?(argument: String, pop: @escaping () -> String?) throws {
         func forcePop() throws -> String {
             guard let value = pop() else { throw OptionParserError.expectedAssociatedValue(argument) }
             return value

Sources/Commands/SwiftPackageTool.swift

@@ -89,7 +89,7 @@ private enum PackageToolFlag: Argument {
     case ignoreDependencies
     case verbose(Int)
 
-    init?(argument: String, pop: () -> String?) throws {
+    init?(argument: String, pop: @escaping () -> String?) throws {
 
         func forcePop() throws -> String {
             guard let value = pop() else { throw OptionParserError.expectedAssociatedValue(argument) }

Sources/Commands/SwiftTestTool.swift

@@ -91,7 +91,7 @@ private enum TestToolFlag: Argument {
     case ignoreDependencies
     case verbose(Int)
 
-    init?(argument: String, pop: () -> String?) throws {
+    init?(argument: String, pop: @escaping () -> String?) throws {
         func forcePop() throws -> String {
             guard let value = pop() else { throw OptionParserError.expectedAssociatedValue(argument) }
             return value