Update as per suggestions

chiragramani · chiragramani · commit 5f5e3dc3bed0 · 2025-11-14T15:48:27.000-08:00
diff --git a/Sources/SwiftDriver/Driver/Driver.swift b/Sources/SwiftDriver/Driver/Driver.swift
@@ -3387,12 +3387,10 @@ extension Driver {
 
     // At most one *use* option allowed
     if providedUse.count > 1 {
-      for i in 0..<(providedUse.count - 1) {
-        for j in (i + 1)..<providedUse.count {
-          diagnosticEngine.emit(
-            .error(Error.conflictingOptions(providedUse[i], providedUse[j])),
-            location: nil
-          )
+      for (i, left) in providedUse.enumerated() {
+        for (j, right) in providedUse.enumerated() {
+          guard i < j else { continue }
+          diagnosticEngine.emit(.error(Error.conflictingOptions(left, right)), location: nil)
         }
       }
     }

Original file line number	Diff line number	Diff line change
`@@ -3387,12 +3387,10 @@ extension Driver {`
`3387`	`3387`
`3388`	`3388`	`// At most one use option allowed`
`3389`	`3389`	`if providedUse.count > 1 {`
`3390`		`- for i in 0..<(providedUse.count - 1) {`
`3391`		`- for j in (i + 1)..<providedUse.count {`
`3392`		`- diagnosticEngine.emit(`
`3393`		`- .error(Error.conflictingOptions(providedUse[i], providedUse[j])),`
`3394`		`- location: nil`
`3395`		`- )`
	`3390`	`+ for (i, left) in providedUse.enumerated() {`
	`3391`	`+ for (j, right) in providedUse.enumerated() {`
	`3392`	`+ guard i < j else { continue }`
	`3393`	`+ diagnosticEngine.emit(.error(Error.conflictingOptions(left, right)), location: nil)`
`3396`	`3394`	`}`
`3397`	`3395`	`}`
`3398`	`3396`	`}`