Skip to content

SE-0334: Pointer API Usability Improvements #39639

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

Merged
merged 8 commits into from
Mar 1, 2022

Conversation

glessard
Copy link
Contributor

@glessard glessard commented Oct 8, 2021

Implementation of SE-0334, "Pointer API Usability Improvements. See proposal here.

Addresses rdar://64342031, SR-11156 (rdar://53272880), rdar://22541346 and rdar://88087964.

@glessard glessard force-pushed the se-pointer-convenience branch from 8bf9ae6 to f3816b7 Compare October 13, 2021 21:11
@glessard glessard force-pushed the se-pointer-convenience branch 2 times, most recently from 30599af to d9c8327 Compare October 22, 2021 22:16
@glessard glessard force-pushed the se-pointer-convenience branch from d9c8327 to e89e66e Compare November 18, 2021 00:21
@glessard glessard force-pushed the se-pointer-convenience branch 2 times, most recently from 4c052ef to 941af40 Compare February 4, 2022 00:01
@glessard glessard changed the title [DNM] Draft implementation for pitch: Pointer API Usability Improvements [draft] SE-0334: Pointer API Usability Improvements Feb 4, 2022
@glessard
Copy link
Contributor Author

glessard commented Feb 4, 2022

@swift-ci please smoke test

@glessard glessard force-pushed the se-pointer-convenience branch from 941af40 to b24ae11 Compare February 4, 2022 19:13
@glessard
Copy link
Contributor Author

glessard commented Feb 4, 2022

@swift-ci please smoke test

@glessard glessard force-pushed the se-pointer-convenience branch from 3751fa6 to 1a06919 Compare February 5, 2022 00:59
@glessard
Copy link
Contributor Author

glessard commented Feb 5, 2022

@swift-ci please smoke test

@glessard glessard marked this pull request as ready for review February 5, 2022 01:00
@glessard glessard changed the title [draft] SE-0334: Pointer API Usability Improvements SE-0334: Pointer API Usability Improvements Feb 5, 2022
@glessard
Copy link
Contributor Author

glessard commented Feb 5, 2022

@swift-ci please test

@swift-ci
Copy link
Contributor

swift-ci commented Feb 5, 2022

Build failed
Swift Test OS X Platform
Git Sha - f3592fb464542edc4f4599e75d593524e6c6d688

@glessard
Copy link
Contributor Author

glessard commented Feb 5, 2022

@swift-ci please smoke test macOS platform

@glessard
Copy link
Contributor Author

glessard commented Feb 5, 2022

@swift-ci please test

@glessard glessard force-pushed the se-pointer-convenience branch 2 times, most recently from 5dd16ac to 3e71a7a Compare February 6, 2022 13:46
@glessard
Copy link
Contributor Author

glessard commented Feb 6, 2022

@swift-ci please smoke test

@glessard
Copy link
Contributor Author

glessard commented Feb 7, 2022

@swift-ci please smoke test macOS platform

@glessard
Copy link
Contributor Author

@swift-ci please smoke test

- The stride check in `UnsafePointer.withMemoryRebound` makes less sense when rebinding memory for a single element.
- This skips the stride-matching portion of the `_debugPrecondition` in `withMemoryRebound` when `count == 1`.
@glessard glessard force-pushed the se-pointer-convenience branch from f51a6e6 to dc09862 Compare February 25, 2022 17:51
@glessard
Copy link
Contributor Author

rebased to resolve merge conflict

@glessard
Copy link
Contributor Author

@swift-ci please smoke test

@glessard glessard force-pushed the se-pointer-convenience branch from dc09862 to bb05e85 Compare February 25, 2022 20:50
@glessard
Copy link
Contributor Author

@swift-ci please smoke test

@glessard
Copy link
Contributor Author

@swift-ci please smoke test macOS platform

@glessard glessard merged commit 577015f into swiftlang:main Mar 1, 2022
@glessard glessard deleted the se-pointer-convenience branch March 1, 2022 20:24
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.

3 participants