-
Notifications
You must be signed in to change notification settings - Fork 10.5k
Use the right key for the image thumbnail dictionaries in the URL resources struct #6689
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
Use the right key for the image thumbnail dictionaries in the URL resources struct #6689
Conversation
resources struct. Fixes rdar://27556178
@swift-ci please test |
Build failed |
@swift-ci please test |
Build failed |
Build failed |
Just to be clear, we're considering this not a source-breaking change because the old code never worked? It still seems like it could break someone's compilation. |
Which "this" are you talking about? Changing the type of the property in the AppKit/UIKit extensions of the URL resources struct or deleting the errant enum? |
Both, really. Someone could have been using an explicit type (wrong as it was) and now it won't compile. Maybe leaving behind an unavailable/renamed typealias would help? |
(It's still not strictly source-compatible, though.) |
I filed another bug for Foundation to figure out the source compat story so I can close out this important fix. 30578582 |
@swift-ci test and merge |
Something funky happened here. I'll restart. Note that we're past our final branch for Swift 3.1. Should this get cherry-picked there as well? @swift-ci Please test |
We don't need this for Swift 3.1. |
The swift importer made a bit of a mess out of the string key here; the real one we want to use is URLThumbnailDictionaryItem.
Fixes rdar://27556178