Skip to content

Commit fb8a592

Browse files
committed
[Concurrency] Rename ExecutorJob back to PartialAsyncTask.
The rename appears to have had some kind of potentially ABI breaking consequence. rdar://141348916
1 parent ff12be1 commit fb8a592

File tree

2 files changed

+11
-1
lines changed

2 files changed

+11
-1
lines changed

stdlib/public/Concurrency/CMakeLists.txt

+1-1
Original file line numberDiff line numberDiff line change
@@ -126,7 +126,7 @@ set(SWIFT_RUNTIME_CONCURRENCY_SWIFT_SOURCES
126126
AsyncThrowingFlatMapSequence.swift
127127
AsyncThrowingMapSequence.swift
128128
AsyncThrowingPrefixWhileSequence.swift
129-
ExecutorJob.swift
129+
PartialAsyncTask.swift
130130
GlobalActor.swift
131131
GlobalConcurrentExecutor.swift
132132
MainActor.swift

stdlib/public/Concurrency/ExecutorJob.swift renamed to stdlib/public/Concurrency/PartialAsyncTask.swift

+10
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,16 @@
1212

1313
import Swift
1414

15+
// N.B. It would be nice to rename this file to ExecutorJob.swift, but when
16+
// trying that we hit an error from the API digester claiming that
17+
//
18+
// +Var UnownedJob.context has mangled name changing from
19+
// 'Swift.UnownedJob.(context in #UNSTABLE ID#) : Builtin.Job' to
20+
// 'Swift.UnownedJob.(context in #UNSTABLE ID#) : Builtin.Job'
21+
//
22+
// This is odd because `context` is private, so it isn't really part of
23+
// the module API.
24+
1525
// TODO: It would be better if some of the functions here could be inlined into
1626
// the Swift code. In particular, some of the ones that deal with data held on
1727
// ExecutorJob.

0 commit comments

Comments
 (0)