Skip to content

Rename internal passRetained helper to _passRetained#126

Merged
lorentey merged 1 commit into
mainfrom
rename-passRetained
Jun 23, 2026
Merged

Rename internal passRetained helper to _passRetained#126
lorentey merged 1 commit into
mainfrom
rename-passRetained

Conversation

@xedin

@xedin xedin commented Jun 23, 2026

Copy link
Copy Markdown
Member

This avoids ambiguity with a public version of the same API that is part of a Swift standard library and returns a non-optional result.

Checklist

  • I've read the Contribution Guidelines
  • My contributions are licensed under the Swift license.
  • I've followed the coding style of the rest of the project.
  • I've added tests covering all new code paths my change adds to the project (if appropriate).
  • I've verified that my change does not break any existing tests.
  • I've updated the documentation if necessary.

Resolves: rdar://180441772

This avoids ambiguity with a public version of the same API
that is part of a Swift standard library and returns a non-optional
result.

@lorentey lorentey left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks! 👍

@lorentey lorentey modified the milestones: 1.3.1, 1.3.0 Jun 23, 2026
@lorentey lorentey merged commit 0442cb5 into main Jun 23, 2026
32 checks passed
@lorentey lorentey deleted the rename-passRetained branch June 23, 2026 21:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants