George review

Author: FranzBusch

Diff for: Sources/NIOCore/EventLoop.swift

@@ -736,7 +736,7 @@ extension EventLoop {
     /// - returns: An `EventLoopFuture` identical to the `EventLoopFuture` returned from `task`.
     @inlinable
     @preconcurrency
-    public func flatSubmit<T: Sendable>(_ task: @escaping @Sendable () -> EventLoopFuture<T>) -> EventLoopFuture<T> { // TODO: This should take a closure that returns fresh
+    public func flatSubmit<T: Sendable>(_ task: @escaping @Sendable () -> EventLoopFuture<T>) -> EventLoopFuture<T> {
         self.submit(task).flatMap { $0 }
     }
 

Diff for: Sources/NIOCore/EventLoopFuture.swift

@@ -184,17 +184,6 @@ public struct EventLoopPromise<Value> {
         self._resolve(value: .success(value))
     }
 
-    /// Deliver a successful result to the associated `EventLoopFuture<Value>` object.
-    ///
-    /// - Note: The call to this method must happen on the same event loop as this promise was created from.
-    ///
-    /// - parameters:
-    ///     - eventLoopBoundValue: The successful result of the operation.
-    @inlinable
-    public func succeed(eventLoopBoundValue: Value) {
-        self._resolve(eventLoopBoundResult: .success(eventLoopBoundValue))
-    }
-
     /// Deliver an error to the associated `EventLoopFuture<Value>` object.
     ///
     /// - parameters:
@@ -247,27 +236,6 @@ public struct EventLoopPromise<Value> {
         self._resolve(value: result)
     }
 
-    /// Complete the promise with the passed in `Result<Value, Error>`.
-    ///
-    /// This method is equivalent to invoking:
-    /// ```
-    /// switch result {
-    /// case .success(let value):
-    ///     promise.succeed(value)
-    /// case .failure(let error):
-    ///     promise.fail(error)
-    /// }
-    /// ```
-    ///
-    /// - Note: The call to this method must happen on the same event loop as this promise was created from.
-    ///
-    /// - parameters:
-    ///     - result: The result which will be used to succeed or fail this promise.
-    @inlinable
-    public func completeWith(eventLoopBoundResult: Result<Value, Error>) {
-        self._resolve(eventLoopBoundResult: eventLoopBoundResult)
-    }
-
     /// Fire the associated `EventLoopFuture` on the appropriate event loop.
     ///
     /// This method provides the primary difference between the `EventLoopPromise` and most
@@ -287,23 +255,6 @@ public struct EventLoopPromise<Value> {
         }
     }
 
-    /// Fire the associated `EventLoopFuture` on the appropriate event loop.
-    ///
-    /// This method provides the primary difference between the `EventLoopPromise` and most
-    /// other `Promise` implementations: specifically, all callbacks fire on the `EventLoop`
-    /// that was used to create the promise.
-    ///
-    /// - Note: The call to this method must happen on the same event loop as this promise was created from.
-    ///
-    /// - parameters:
-    ///     - value: The value to fire the future with.
-    @inlinable
-    internal func _resolve(eventLoopBoundResult: Result<Value, Error>) {
-        self.futureResult.eventLoop.assertInEventLoop()
-
-        self._setValue(value: eventLoopBoundResult)._run()
-    }
-
     /// Set the future result and get the associated callbacks.
     ///
     /// - parameters: