Merge pull request #14952 from PycKamil/remove-obsolete-check-in-DriverUtils.swift

swift-ci · web-flow · commit 26c08fb34642 · 2018-03-03T17:10:37.000-08:00
diff --git a/benchmark/utils/DriverUtils.swift b/benchmark/utils/DriverUtils.swift
@@ -208,14 +208,10 @@ struct TestConfig {
     }
 
     if let x = benchArgs.optionalArgsMap["--sleep"] {
-      if x.isEmpty {
+      guard let v = Int(x) else {
         return .fail("--sleep requires a non-empty integer value")
       }
-      let v: Int? = Int(x)
-      if v == nil {
-        return .fail("--sleep requires a non-empty integer value")
-      }
-      afterRunSleep = v!
+      afterRunSleep = v
     }
 
     if let _ = benchArgs.optionalArgsMap["--list"] {

Original file line number	Diff line number	Diff line change
`@@ -208,14 +208,10 @@ struct TestConfig {`
`208`	`208`	`}`
`209`	`209`
`210`	`210`	`if let x = benchArgs.optionalArgsMap["--sleep"] {`
`211`		`- if x.isEmpty {`
	`211`	`+ guard let v = Int(x) else {`
`212`	`212`	`return .fail("--sleep requires a non-empty integer value")`
`213`	`213`	`}`
`214`		`- let v: Int? = Int(x)`
`215`		`- if v == nil {`
`216`		`- return .fail("--sleep requires a non-empty integer value")`
`217`		`- }`
`218`		`- afterRunSleep = v!`
	`214`	`+ afterRunSleep = v`
`219`	`215`	`}`
`220`	`216`
`221`	`217`	`if let _ = benchArgs.optionalArgsMap["--list"] {`