Skip to content

Commit 1ea91fa

Browse files
committed
Remove the last remaining NIOAtomic
Motivation Warnings aren't great, and NIOAtomic is deprecated. Modifications Replace the last use of NIOAtomic with ManagedAtomic. Result Fewer warnings Fixes swift-server#606
1 parent 2adca4b commit 1ea91fa

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

Diff for: Sources/AsyncHTTPClient/HTTPClient.swift

+3-2
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@
1313
//===----------------------------------------------------------------------===//
1414

1515
import Foundation
16+
import Atomics
1617
import Logging
1718
import NIOConcurrencyHelpers
1819
import NIOCore
@@ -36,7 +37,7 @@ extension Logger {
3637
}
3738
}
3839

39-
let globalRequestID = NIOAtomic<Int>.makeAtomic(value: 0)
40+
let globalRequestID = ManagedAtomic(0)
4041

4142
/// HTTPClient class provides API for request execution.
4243
///
@@ -541,7 +542,7 @@ public class HTTPClient {
541542
logger originalLogger: Logger?,
542543
redirectState: RedirectState?
543544
) -> Task<Delegate.Response> {
544-
let logger = (originalLogger ?? HTTPClient.loggingDisabled).attachingRequestInformation(request, requestID: globalRequestID.add(1))
545+
let logger = (originalLogger ?? HTTPClient.loggingDisabled).attachingRequestInformation(request, requestID: globalRequestID.wrappingIncrementThenLoad(ordering: .relaxed))
545546
let taskEL: EventLoop
546547
switch eventLoopPreference.preference {
547548
case .indifferent:

0 commit comments

Comments
 (0)