Don't force unwrap – use fatalError with a message instead

dfed · dfed · commit 6e0519cb0017 · 2022-11-22T22:59:35.000-08:00
diff --git a/Sources/AsyncQueue/AsyncQueue.swift b/Sources/AsyncQueue/AsyncQueue.swift
@@ -32,6 +32,9 @@ public final class AsyncQueue: Sendable {
         let taskStream = AsyncStream<@Sendable () async -> Void> { continuation in
             capturedTaskStreamContinuation = continuation
         }
+        guard let capturedTaskStreamContinuation = capturedTaskStreamContinuation else {
+            fatalError("Continuation not captured during stream creation!")
+        }
         taskStreamContinuation = capturedTaskStreamContinuation
 
         streamTask = Task.detached(priority: priority) {
@@ -85,5 +88,5 @@ public final class AsyncQueue: Sendable {
     // MARK: Private
 
     private let streamTask: Task<Void, Never>
-    private let taskStreamContinuation: AsyncStream<@Sendable () async -> Void>.Continuation!
+    private let taskStreamContinuation: AsyncStream<@Sendable () async -> Void>.Continuation
 }

Original file line number	Diff line number	Diff line change
`@@ -32,6 +32,9 @@ public final class AsyncQueue: Sendable {`
`32`	`32`	`let taskStream = AsyncStream<@Sendable () async -> Void> { continuation in`
`33`	`33`	`capturedTaskStreamContinuation = continuation`
`34`	`34`	`}`
	`35`	`+ guard let capturedTaskStreamContinuation = capturedTaskStreamContinuation else {`
	`36`	`+ fatalError("Continuation not captured during stream creation!")`
	`37`	`+ }`
`35`	`38`	`taskStreamContinuation = capturedTaskStreamContinuation`
`36`	`39`
`37`	`40`	`streamTask = Task.detached(priority: priority) {`
`@@ -85,5 +88,5 @@ public final class AsyncQueue: Sendable {`
`85`	`88`	`// MARK: Private`
`86`	`89`
`87`	`90`	`private let streamTask: Task<Void, Never>`
`88`		`- private let taskStreamContinuation: AsyncStream<@Sendable () async -> Void>.Continuation!`
	`91`	`+ private let taskStreamContinuation: AsyncStream<@Sendable () async -> Void>.Continuation`
`89`	`92`	`}`