Skip to content

Roll back a portion of SE-0110 #10414

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 2 commits into from
Jun 21, 2017
Merged

Roll back a portion of SE-0110 #10414

merged 2 commits into from
Jun 21, 2017

Conversation

rudkx
Copy link
Contributor

@rudkx rudkx commented Jun 20, 2017

As described in:

https://lists.swift.org/pipermail/swift-evolution-announce/2017-June/000386.html

Specifically allow an N-ary argument function to be passed as an
argument in a place where a function of a single N-tuple is expected.

Fixes: rdar://problem/32875953
(cherry picked from commit 0ba4a31)

@rudkx
Copy link
Contributor Author

rudkx commented Jun 20, 2017

@swift-ci Please test

@rudkx rudkx requested a review from DougGregor June 20, 2017 20:44
@swift-ci
Copy link
Contributor

Build failed
Jenkins build - Swift Test Linux Platform
Git Commit - 7b17ab0c1f7cbbac288dc183f9627ce793611426
Test requested by - @rudkx

rudkx added 2 commits June 20, 2017 14:06
As described in:
  https://lists.swift.org/pipermail/swift-evolution-announce/2017-June/000386.html

Specifically allow an N-ary argument function to be passed as an
argument in a place where a function of a single N-tuple is expected.

Fixes: rdar://problem/32875953
(cherry picked from commit 0ba4a31)
@rudkx
Copy link
Contributor Author

rudkx commented Jun 20, 2017

@swift-ci Please test

@swift-ci
Copy link
Contributor

Build failed
Jenkins build - Swift Test Linux Platform
Git Commit - 7b17ab0c1f7cbbac288dc183f9627ce793611426
Test requested by - @rudkx

@swift-ci
Copy link
Contributor

Build failed
Jenkins build - Swift Test OS X Platform
Git Commit - 7b17ab0c1f7cbbac288dc183f9627ce793611426
Test requested by - @rudkx

@slavapestov
Copy link
Contributor

Is this going into master as well?

@rudkx
Copy link
Contributor Author

rudkx commented Jun 20, 2017

Already there: 706922d

Copy link
Member

@DougGregor DougGregor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, and thank you for pulling in the examples from my e-mail.

@rudkx rudkx added this to the Swift 4.0 milestone Jun 20, 2017
@tkremenek
Copy link
Member

@swift-ci test source compatibility

1 similar comment
@ematejska
Copy link
Contributor

@swift-ci test source compatibility

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.

6 participants