swift-server / async-http-client

HTTP client library built on SwiftNIO

https://swiftpackageindex.com/swift-server/async-http-client/main/documentation/asynchttpclient

HTTPClient is still using NIOAtomic

adam-fowler opened this issue 2 years ago · comments

Adam Fowler commented 2 years ago

Given the other NIOAtomics were replaced with ManagedAtomic I assume the HTTPClient one will want to be replaced as well

async-http-client/Sources/AsyncHTTPClient/HTTPClient.swift

Line 39 in 2adca4b

let globalRequestID = NIOAtomic<Int>.makeAtomic(value: 0)

Cory Benfield commented 2 years ago

Agreed.

Cory Benfield commented 2 years ago

Looks like this was just an oversight in #603, I'll quickly clean it up.

Cory Benfield commented 2 years ago

Shipped in 1.11.3.