Skip to content

Commit a8a8bdc

Browse files
authored
Merge pull request #20940 from compnerd/threading-cleanup
SwiftPrivateThreadExtras: clean up using WinSDK SDK overlay
2 parents 1791edb + 98d71ff commit a8a8bdc

File tree

3 files changed

+4
-7
lines changed

3 files changed

+4
-7
lines changed

stdlib/private/SwiftPrivateThreadExtras/CMakeLists.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ add_swift_target_library(swiftSwiftPrivateThreadExtras ${SWIFT_STDLIB_LIBRARY_BU
1212
SWIFT_MODULE_DEPENDS_FREEBSD Glibc
1313
SWIFT_MODULE_DEPENDS_CYGWIN Glibc
1414
SWIFT_MODULE_DEPENDS_HAIKU Glibc
15-
SWIFT_MODULE_DEPENDS_WINDOWS MSVCRT
15+
SWIFT_MODULE_DEPENDS_WINDOWS MSVCRT WinSDK
1616
SWIFT_COMPILE_FLAGS
1717
INSTALL_IN_COMPONENT stdlib-experimental)
1818

stdlib/private/SwiftPrivateThreadExtras/SwiftPrivateThreadExtras.swift

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -116,10 +116,8 @@ public func _stdlib_thread_join<Result>(
116116
_ resultType: Result.Type
117117
) -> (CInt, Result?) {
118118
#if os(Windows)
119-
// TODO(compnerd) modularize rpc.h for INFINITE (0xffffffff)
120-
let result = WaitForSingleObject(thread, 0xffffffff);
121-
// TODO(compnerd) modularize WinBase.h for WAIT_OBJECT_0 (0)
122-
if result == 0 {
119+
let result = WaitForSingleObject(thread, INFINITE)
120+
if result == WAIT_OBJECT_0 {
123121
var threadResult: DWORD = 0
124122
GetExitCodeThread(thread, &threadResult)
125123
CloseHandle(thread)

stdlib/private/SwiftPrivateThreadExtras/ThreadBarriers.swift

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -122,9 +122,8 @@ public func _stdlib_thread_barrier_wait(
122122
if barrier.pointee.numThreadsWaiting < barrier.pointee.count {
123123
// Put the thread to sleep.
124124
#if os(Windows)
125-
// TODO(compnerd) modularize rpc.h to get INFIITE (0xffffffff)
126125
if SleepConditionVariableSRW(barrier.pointee.cond!, barrier.pointee.mutex!,
127-
0xffffffff, 0) == 0 {
126+
INFINITE, 0) == 0 {
128127
return -1
129128
}
130129
ReleaseSRWLockExclusive(barrier.pointee.mutex!)

0 commit comments

Comments
 (0)