Skip to content

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

Merged

Conversation

parkera
Copy link
Contributor

@parkera parkera commented Jan 9, 2017

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

@parkera
Copy link
Contributor Author

parkera commented Jan 9, 2017

@swift-ci please test

@swift-ci
Copy link
Contributor

swift-ci commented Jan 9, 2017

Build failed
Jenkins build - Swift Test OS X Platform
Git Commit - 1553cde
Test requested by - @parkera

@parkera
Copy link
Contributor Author

parkera commented Jan 10, 2017

@swift-ci please test

@swift-ci
Copy link
Contributor

Build failed
Jenkins build - Swift Test OS X Platform
Git Commit - 1553cde
Test requested by - @parkera

@swift-ci
Copy link
Contributor

Build failed
Jenkins build - Swift Test Linux Platform
Git Commit - 1553cde
Test requested by - @parkera

@jrose-apple
Copy link
Contributor

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.

@parkera
Copy link
Contributor Author

parkera commented Jan 10, 2017

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?

@jrose-apple
Copy link
Contributor

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?

@jrose-apple
Copy link
Contributor

(It's still not strictly source-compatible, though.)

@parkera
Copy link
Contributor Author

parkera commented Feb 17, 2017

I filed another bug for Foundation to figure out the source compat story so I can close out this important fix. 30578582

@parkera
Copy link
Contributor Author

parkera commented Feb 17, 2017

@swift-ci test and merge

@jrose-apple
Copy link
Contributor

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

@parkera
Copy link
Contributor Author

parkera commented Feb 20, 2017

We don't need this for Swift 3.1.

@jrose-apple jrose-apple merged commit 20bdedf into swiftlang:master Feb 21, 2017
@parkera parkera deleted the pr27556178_wrong_image_thumbnail_key branch February 21, 2017 19:09
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.

3 participants