Skip to content

Sema: Fix another SE-0110 issue #9454

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
May 10, 2017

Conversation

slavapestov
Copy link
Contributor

The formal type of methods should be (Type) -> (Args...) -> (),
not Type -> (Args...) -> ().

This only matters in Swift 4 mode, where it was preventing the
newly-added test case from type checking.

Fixes rdar://problem/31725325.

@slavapestov
Copy link
Contributor Author

@swift-ci Please smoke test

The formal type of methods should be (Type) -> (Args...) -> (),
not Type -> (Args...) -> ().

This only matters in Swift 4 mode, where it was preventing the
newly-added test case from type checking.

Fixes <rdar://problem/31725325>.
@slavapestov slavapestov force-pushed the se-0110-strikes-again branch from 2f54ee8 to f7bf7ac Compare May 10, 2017 05:28
@slavapestov
Copy link
Contributor Author

@swift-ci Please smoke test

1 similar comment
@slavapestov
Copy link
Contributor Author

@swift-ci Please smoke test

@slavapestov
Copy link
Contributor Author

@swift-ci Please test source compatibility

1 similar comment
@slavapestov
Copy link
Contributor Author

@swift-ci Please test source compatibility

@slavapestov slavapestov merged commit 243b91a into swiftlang:master May 10, 2017
huonw added a commit to huonw/swift that referenced this pull request Jun 14, 2017
Method types are now required to be `(Self) -> (Args...) -> Return`, not
`Self -> (Args...) -> Return`.

See also swiftlang#9454 .

Fixes rdar://problem/32588152 .
huonw added a commit to huonw/swift that referenced this pull request Jun 14, 2017
Method types are now required to be `(Self) -> (Args...) -> Return`, not
`Self -> (Args...) -> Return`.

See also swiftlang#9454 .

Fixes rdar://problem/32588152 .
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