review fixes

Author: artemredkin

Sources/AsyncHTTPClient/HTTPHandler.swift

@@ -841,12 +841,12 @@ extension TaskHandler: ChannelDuplexHandler {
                 let promise = self.task.eventLoop.makePromise(of: Void.self)
                 // All writes have to be switched to the channel EL if channel and task ELs differ
                 if context.eventLoop.inEventLoop {
-                    context.writeAndFlush(self.wrapOutboundOut(.body(part)), promise: promise)
                     self.actualBodyLength += part.readableBytes
+                    context.writeAndFlush(self.wrapOutboundOut(.body(part)), promise: promise)
                 } else {
                     context.eventLoop.execute {
-                        context.writeAndFlush(self.wrapOutboundOut(.body(part)), promise: promise)
                         self.actualBodyLength += part.readableBytes
+                        context.writeAndFlush(self.wrapOutboundOut(.body(part)), promise: promise)
                     }
                 }
 

Tests/AsyncHTTPClientTests/HTTPClientTestUtils.swift

@@ -361,9 +361,6 @@ internal struct HTTPResponseBuilder {
     }
 }
 
-// let globalRequestCounter = NIOAtomic<Int>.makeAtomic(value: 0)
-// let globalConnectionCounter = NIOAtomic<Int>.makeAtomic(value: 0)
-
 internal struct RequestInfo: Codable {
     var data: String
     var requestNumber: Int