Skip to content

Conversation

weissi
Copy link
Member

@weissi weissi commented Jul 11, 2025

Motivation:

Structured Concurrency is helpful but so far, MTELG was very difficult to use with it mostly because Swift is still missing async in defers.

This builds on

Modifications:

  • Add MTELG.withELG { ... }

Result:

MTELG + SC = <3

@weissi weissi requested review from Lukasa and FranzBusch July 11, 2025 10:10
@weissi weissi force-pushed the jw-sc-mtelg branch 4 times, most recently from ed2df3f to a0c8b6c Compare July 11, 2025 12:01
@Lukasa Lukasa added the 🆕 semver/minor Adds new public API. label Jul 11, 2025
@weissi weissi force-pushed the jw-sc-mtelg branch 2 times, most recently from 100719b to b2bb2d0 Compare July 11, 2025 15:11
@weissi weissi requested a review from Lukasa July 11, 2025 15:11
@weissi weissi marked this pull request as draft July 11, 2025 15:11
@weissi
Copy link
Member Author

weissi commented Jul 11, 2025

I will split this PR in two:

  • the NIOThread refactor
  • the actual new functionality

@weissi weissi marked this pull request as ready for review July 15, 2025 09:48
@weissi weissi merged commit 933b9d7 into apple:main Jul 15, 2025
41 checks passed
@weissi weissi deleted the jw-sc-mtelg branch July 15, 2025 11:14
zaneenders pushed a commit to zaneenders/swift-nio that referenced this pull request Jul 23, 2025
### Motivation:

Structured Concurrency is helpful but so far, MTELG was very difficult
to use with it mostly because Swift is still missing `async` in
`defer`s.

This builds on
- apple#3297 
- apple#3302
- apple#3304

### Modifications:

- Add MTELG.withELG { ... }

### Result:

MTELG + SC = <3
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🆕 semver/minor Adds new public API.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants