Adopt ‘as’ bridging on Linux. #1526
Conversation
| } | ||
|
|
||
| extension AffineTransform : _ObjectTypeBridgeable { | ||
| extension AffineTransform : _ObjectiveCBridgeable { |
There was a problem hiding this comment.
The main change is removing the parallel type _ObjectTypeBridgeable and replacing it with the newly-exposed-on-Linux _ObjectiveCBridgeable.
| } | ||
|
|
||
| public // COMPILER_INTRINSIC | ||
| func _convertNSErrorToError(_ error: NSError?) -> Error { |
There was a problem hiding this comment.
Converting Error to NSError and back is in scope, but support is WIP.
|
Ultimately we should also change
|
millenomi
changed the title
|
Looks good! |
|
@swift-ci please test and merge |
|
The corresponding compiler change in had to be reverted. |
|
I canceled the "and merge" build in CI so that this wouldn't be automatically merged. I think the underlying linux test will continue though. |
|
On hold as we figure out the 14.04 issues on the compiler patch. |
|
Please test with the following: @swift-ci please test |
|
Please test with the following: @swift-ci please test |
|
Please test with the following: @swift-ci please test |
|
Simultaneous merging with swiftlang/swift#16736 imminent… |
| return val._bridgeToAnyObject() as! NSObject | ||
| } else { | ||
| return _SwiftValue(value) | ||
| return (value as AnyObject) as! NSObject |
There was a problem hiding this comment.
Looks like this change leads to crash in cases like this:
class A {}
let array: NSArray = [A()]
There was a problem hiding this comment.
@millenomi Can you look into this https://github.com/apple/swift-corelibs-foundation/pull/2500/files ?
4 participants
This patch adopts 'as' bridging on Linux. It requires swiftlang/swift#16022 — see that PR for more details.