-
Notifications
You must be signed in to change notification settings - Fork 10.5k
[overlay] Build CoreGraphics overlay in Swift 4 mode #19598
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
Conversation
@swift-ci Please test macOS platform |
// known to Swift as CGColor(red:green:blue:alpha:). Unfortunately this API | ||
// is not available on platforms other than macOS. It would be possible to | ||
// replicate the exact functionality of that API using | ||
// kGColorSpaceGenericRGB, but it is marked as unavailable for Swift. The |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think we should change the behavior on macOS, though; that should continue using the generic init(red:green:blue:alpha:)
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(unless the CoreGraphics folks want it to be consistent across platforms)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, knowing nothing about color spaces etc., I asked them for a review. Calling the old one for macOS is a good idea, and I can even do that with if #available
, thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah, #available
won't do that, since it treats platforms you don't mention as "true". You'll still want #if
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
😞
Resolves: <rdar://problem/35338851>
bbdbb16
to
8168fb8
Compare
@swift-ci Please test macOS platform |
Build failed |
@swift-ci Please smoke test Linux platform |
Resolves: rdar://problem/35338851