Skip to content

UnsafePointer conversion fixes. #487

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 1 commit into from
Jul 29, 2016
Merged

UnsafePointer conversion fixes. #487

merged 1 commit into from
Jul 29, 2016

Conversation

atrick
Copy link
Contributor

@atrick atrick commented Jul 28, 2016

Update for SE-0107: UnsafeRawPointer.

Some of this is pretty rough, but it's necessary to
keep the build working and land the raw vs. typed pointer feature.

As expected there are two major pain points:

  1. CString vs. UTF8 types
  2. NSData and company use pointer to UInt8 for untyped memory.

@atrick
Copy link
Contributor Author

atrick commented Jul 28, 2016

@parkera This is just a stop-gap to allow the build to succeed when we're ready to pull in the stdlib changes sometime today. I plan to go through over all these changes again later and find a cleaner way to handle things.

@atrick
Copy link
Contributor Author

atrick commented Jul 28, 2016

I'll update this after
swiftlang/swift#3832

Also, I noticed a stdlib dependency which hasn't been checked in yet, so this should not be merged.

@atrick atrick changed the title UnsafePointer conversion fixes. [WIP] UnsafePointer conversion fixes. Jul 28, 2016
@parkera
Copy link
Contributor

parkera commented Jul 28, 2016

Ok.

@atrick atrick force-pushed the rawptr branch 2 times, most recently from e0a6c0c to 9d77896 Compare July 29, 2016 06:55
Update for SE-0107: UnsafeRawPointer.

Some of this is pretty rough, but it's necessary to
keep the build working and land the raw vs. typed pointer feature.

As expected there are two major pain points:

1. CString vs. UTF8 types

2. NSData and company use pointer to UInt8 for untyped memory.
@atrick
Copy link
Contributor Author

atrick commented Jul 29, 2016

This passes 'build-script -R --llbuild --foundation --xctest -T' with the latest swift master.
I need this to proceed with changes on master.
Merging and keeping an eye on the bots.

@atrick atrick changed the title [WIP] UnsafePointer conversion fixes. UnsafePointer conversion fixes. Jul 29, 2016
@atrick atrick merged commit 007089a into swiftlang:master Jul 29, 2016
atrick added a commit that referenced this pull request Jul 29, 2016
This reverts commit 965c6af.

This reapplies pointer conversion fixed from PR #487.

The previous failure turned out to be a spurious device error.
phausler pushed a commit that referenced this pull request Aug 4, 2016
This reverts commit 965c6af.

This reapplies pointer conversion fixed from PR #487.

The previous failure turned out to be a spurious device error.
kateinoigakukun pushed a commit to kateinoigakukun/swift-corelibs-foundation that referenced this pull request Oct 11, 2023
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