Skip to content

[stdlib] rewrite parameter of copyBytes using a primary associated type #61088

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

glessard
Copy link
Contributor

@glessard glessard commented Sep 14, 2022

Express the generic parameter of UnsafeMutableRawBufferPointer.copyBytes() function using some notation
(primary associated type).

Addresses rdar://99933149

@glessard glessard changed the title [stdlib] use a primary associated type [stdlib] rewrite parameter of copyBytes using a primary associated type Sep 14, 2022
@glessard
Copy link
Contributor Author

glessard commented Sep 14, 2022

Necessitates a workaround for rdar://99932758 (issue with test/api-digester/stability-stdlib-source.swift)
Added an exception in commit 322771d that should not be needed.

@glessard
Copy link
Contributor Author

@swift-ci please test macOS platform

@glessard
Copy link
Contributor Author

glessard commented Jan 3, 2024

@swift-ci please test

@glessard glessard force-pushed the copyBytes-primary-associated-types branch from 322771d to 2e01a25 Compare January 3, 2024 23:34
@glessard glessard requested review from phausler and Azoy January 5, 2024 01:20
@glessard
Copy link
Contributor Author

glessard commented Jan 5, 2024

Given the unresolved issue with the api-digester test, should we go ahead with this, or nix it?

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.

1 participant