Skip to content

Conversation

@Lukasa
Copy link
Contributor

@Lukasa Lukasa commented Jun 20, 2025

Motivation

The Swift 6 language mode adopts data-race safety by default. While Swift Crypto itself has no concurrent code, it is still useful to force us to ensure that our code is properly Sendable-correct.

Modifications

  • @unchecked Sendable on several CoW data types
  • Some necessary Sendable constraints on ECToolbox protocols
  • Add some missing protocol constraints on ARC types.

Result

Swift 6 clean.

Motivation

The Swift 6 language mode adopts data-race safety by default. While
Swift Crypto itself has no concurrent code, it is still useful to
force us to ensure that our code is properly Sendable-correct.

Modifications

- @unchecked Sendable on several CoW data types
- Some necessary Sendable constraints on ECToolbox protocols
- Add some missing protocol constraints on ARC types.

Result

Swift 6 clean.
@Lukasa Lukasa added the 🆕 semver/minor Adds new public API. label Jun 20, 2025
Copy link
Contributor

@glbrntt glbrntt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. You might want to switch off the 5.10 jobs as well.

@Lukasa Lukasa merged commit 9b06971 into apple:wwdc-25 Jun 23, 2025
28 of 30 checks passed
@Lukasa Lukasa deleted the cb-swift-6-mode branch June 23, 2025 08:50
This was referenced Sep 24, 2025
Lukasa added a commit that referenced this pull request Sep 26, 2025
This change updates Crypto to WWDC25 equivalent code.

Each of these changes were individually merged to the WWDC25 branch.
This includes:

- #359
- #363
- #365
- #367
- #368
- #397
- #403
- #410
- #407
- #401
- #415

---------

Co-authored-by: YourMJK <[email protected]>
Co-authored-by: Si Beaumont <[email protected]>
Co-authored-by: Rick Newton-Rogers <[email protected]>
Co-authored-by: Raphael <[email protected]>
Co-authored-by: Evan Wilde <[email protected]>
Co-authored-by: Tim Condon <[email protected]>
Co-authored-by: Gus Cairo <[email protected]>
Co-authored-by: George Barnett <[email protected]>
Co-authored-by: Fabrice de Gans <[email protected]>
Co-authored-by: Wojciech Nagrodzki <[email protected]>
Co-authored-by: Jesse L. Zamora <[email protected]>
Co-authored-by: aryan-25 <[email protected]>
Co-authored-by: Gwynne Raskind <[email protected]>
Co-authored-by: 3405691582 <[email protected]>
Co-authored-by: Paul Toffoloni <[email protected]>
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