Skip to content

[Migrator] Add Swift. prefix to type(of:) expressions in Swift 3 #9348

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
May 6, 2017
Merged

[Migrator] Add Swift. prefix to type(of:) expressions in Swift 3 #9348

merged 1 commit into from
May 6, 2017

Conversation

bitjammer
Copy link
Contributor

This changed to be resolved by overload resolution in Swift 4, and so
can suddenly be shadowed by similarly named properties and functions
that are visible at the expression's location. When in Swift 3 mode, if
there is a visible declaration named "type", add the Swift. prefix to
disambiguate in Swift 4 mode.

rdar://problem/31997321

@bitjammer
Copy link
Contributor Author

@swift-ci Please smoke test

@jckarter
Copy link
Contributor

jckarter commented May 5, 2017

Thanks!

@bitjammer
Copy link
Contributor Author

Huh! This has no effect on Linux?

@bitjammer
Copy link
Contributor Author

@akyrtzi reports that he also saw that an unrelated test wasn't printing the migrated content either, so it sounds like something else is going on. I'll investigate in a separate PR.

@bitjammer
Copy link
Contributor Author

bitjammer commented May 5, 2017

Figured out the problem there, will address after this.
rdar://problem/32025974

@bitjammer
Copy link
Contributor Author

@swift-ci Please smoke test

This changed to be resolved by overload resolution in Swift 4, and so
can suddenly be shadowed by similarly named properties and functions
that are visible at the expression's location. When in Swift 3 mode, if
there is a visible declaration named "type", add the `Swift.` prefix to
disambiguate in Swift 4 mode.

rdar://problem/31997321
@bitjammer
Copy link
Contributor Author

@swift-ci Please smoke test

@bitjammer bitjammer merged commit 20a429b into swiftlang:master May 6, 2017
@bitjammer bitjammer deleted the rdar-add-Swift-prefix-to-typeof branch May 6, 2017 01:14
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