Skip to content

Fix EncodingError/DecodingError bridging to NSError #10674

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

Closed

Conversation

itaiferber
Copy link
Contributor

What's in this pull request?
Fixes bridging of these error types to NSError and LocalizedError. CustomNSError conformance only enables bridging for types which are declared in the same module as the conformance. Since we cannot use CustomNSError in the stdlib, we need to use other entry points to allow this bridging.

CustomNSError conformance only enables bridging for types which are
declared in the same module as the conformance. Since we cannot use
CustomNSError in the stdlib, we need to use other entry points to allow
this bridging.
@itaiferber itaiferber requested a review from phausler June 28, 2017 19:38
@itaiferber
Copy link
Contributor Author

@swift-ci Please test macOS

@jrose-apple jrose-apple requested a review from DougGregor June 28, 2017 20:10
@itaiferber
Copy link
Contributor Author

@swift-ci Please smoke test Linux

@itaiferber
Copy link
Contributor Author

Whoops, I just realized I'd pushed the wrong version of this. Rebasing and repushing momentarily.

@itaiferber
Copy link
Contributor Author

Actually, going to reopen this as a new PR with a different set of changes.

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