Skip to content

Commit 2b05dc6

Browse files
authored
Merge pull request #82833 from xedin/remove-startSynchronously
[Concurrency] Remove deprecated `Task.startSynchronously` API
2 parents 2c1b596 + e108524 commit 2b05dc6

File tree

4 files changed

+2
-32
lines changed

4 files changed

+2
-32
lines changed

stdlib/public/Concurrency/Task+immediate.swift.gyb

Lines changed: 0 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -33,25 +33,6 @@ import Swift
3333
@available(SwiftStdlib 6.2, *)
3434
extension Task where Failure == ${FAILURE_TYPE} {
3535

36-
// FIXME: This method is left in place to give adopters time to switch to `immediate` but it's going
37-
// to be removed soon, since this spelling was rejected as part of SE-0472 proposal.
38-
@available(SwiftStdlib 6.2, *)
39-
@available(*, deprecated, renamed: "immediate")
40-
// Used to preserve the symbols as originally declared without `@isolated(any)` attribute on `operation:`.
41-
% if FAILURE_TYPE == "Error":
42-
@_silgen_name("$sScTss5Error_pRs_rlE18startSynchronously4name8priority_ScTyxsAA_pGSSSg_ScPSgxyYaKcntFZ")
43-
% elif FAILURE_TYPE == "Never":
44-
@_silgen_name("$sScTss5NeverORs_rlE18startSynchronously4name8priority_ScTyxABGSSSg_ScPSgxyYaKcntFZ")
45-
% end
46-
@discardableResult
47-
public static func startSynchronously(
48-
name: String? = nil,
49-
priority: TaskPriority? = nil,
50-
@_implicitSelfCapture @_inheritActorContext(always) _ operation: sending @isolated(any) @escaping () async ${THROWS} -> Success
51-
) -> Task<Success, ${FAILURE_TYPE}> {
52-
immediate(name: name, priority: priority, operation: operation)
53-
}
54-
5536
/// Create and immediately start running a new task in the context of the calling thread/task.
5637
///
5738
/// This function _starts_ the created task on the calling context.

test/Concurrency/Runtime/startImmediately.swift

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -436,11 +436,6 @@ print("call_startSynchronously_insideActor()")
436436

437437
actor A {
438438
func f() {
439-
Task.startSynchronously(name: "hello") { print("Task.startSynchronously (\(Task.name!))") }
440-
Task.startSynchronously() { print("Task.startSynchronously") }
441-
}
442-
443-
func f2() {
444439
Task.immediate(name: "hello") { print("Task.immediate (\(Task.name!))") }
445440
Task.immediate() { print("Task.immediate") }
446441

@@ -451,12 +446,10 @@ actor A {
451446

452447
func call_startSynchronously_insideActor() async {
453448
await A().f()
454-
await A().f2()
455449
}
456450

457451
await call_startSynchronously_insideActor()
458452

459453
// CHECK-LABEL: call_startSynchronously_insideActor()
460-
// Those two definitely in this order, however the startSynchronously is not determinate
461454
// CHECK: Task.immediate
462455
// CHECK: Task.immediate { @MainActor }

test/abi/macOS/arm64/concurrency.swift

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -389,10 +389,8 @@ Added: _$sScTss5NeverORszABRs_rlE4nameSSSgvgZ
389389
Added: _$sScTss5NeverORszABRs_rlE4nameSSSgvpZMV
390390
Added: _swift_task_getCurrentTaskName
391391

392-
// startSynchronously, immediate, addImmediateTask{UnlessCancelled}
392+
// immediate, addImmediateTask{UnlessCancelled}
393393
Added: _swift_task_immediate
394-
Added: _$sScTss5Error_pRs_rlE18startSynchronously4name8priority_ScTyxsAA_pGSSSg_ScPSgxyYaKcntFZ
395-
Added: _$sScTss5NeverORs_rlE18startSynchronously4name8priority_ScTyxABGSSSg_ScPSgxyYaKcntFZ
396394

397395
// isIsolatingCurrentContext
398396
Added: _swift_task_invokeSwiftIsIsolatingCurrentContext

test/abi/macOS/x86_64/concurrency.swift

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -389,10 +389,8 @@ Added: _$sScTss5NeverORszABRs_rlE4nameSSSgvgZ
389389
Added: _$sScTss5NeverORszABRs_rlE4nameSSSgvpZMV
390390
Added: _swift_task_getCurrentTaskName
391391

392-
// startSynchronously, immediate, addImmediateTask{UnlessCancelled}
392+
// immediate, addImmediateTask{UnlessCancelled}
393393
Added: _swift_task_immediate
394-
Added: _$sScTss5Error_pRs_rlE18startSynchronously4name8priority_ScTyxsAA_pGSSSg_ScPSgxyYaKcntFZ
395-
Added: _$sScTss5NeverORs_rlE18startSynchronously4name8priority_ScTyxABGSSSg_ScPSgxyYaKcntFZ
396394

397395
// isIsolatingCurrentContext
398396
Added: _swift_task_invokeSwiftIsIsolatingCurrentContext

0 commit comments

Comments
 (0)