Skip to content

Don't flatten input types in ObjC representation check #17123

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
Jun 13, 2018

Conversation

CodaFi
Copy link
Contributor

@CodaFi CodaFi commented Jun 11, 2018

Arguments of tuple type are not representable in Objective-C but the
old check was looking specifically for them.

Let's see what this breaks

@CodaFi CodaFi requested a review from DougGregor June 11, 2018 22:46
@CodaFi
Copy link
Contributor Author

CodaFi commented Jun 11, 2018

@swift-ci please test source compatibility

@slavapestov
Copy link
Contributor

Looks like you can special case (Void) -> Void in Swift 3 mode.

Arguments of tuple type are not representable in Objective-C but the
old check was looking specifically for them.
@CodaFi CodaFi force-pushed the unexpected-parentheticals branch from d3f4a00 to e15e293 Compare June 12, 2018 22:13
@CodaFi
Copy link
Contributor Author

CodaFi commented Jun 12, 2018

@swift-ci please test source compatibility

@CodaFi
Copy link
Contributor Author

CodaFi commented Jun 13, 2018

Praise be, praise be

@swift-ci please smoke test

@CodaFi
Copy link
Contributor Author

CodaFi commented Jun 13, 2018

⛵️

@CodaFi CodaFi merged commit 5672c69 into swiftlang:master Jun 13, 2018
@CodaFi CodaFi deleted the unexpected-parentheticals branch June 13, 2018 01:50
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