Skip to content

Conversation

aschwaighofer
Copy link
Contributor

We claim to handle optional input types but not all paths of this function
actually do.

rdar://39195672

(cherry picked from #15772)

We claim to handle optional input types but not all paths of this function
actually do.

rdar://39195672

(cherry picked from swiftlang#15772)
@aschwaighofer
Copy link
Contributor Author

@swift-ci Please test

@swift-ci
Copy link
Contributor

swift-ci commented Apr 6, 2018

Build failed
Swift Test Linux Platform
Git Sha - 542fdab

@swift-ci
Copy link
Contributor

swift-ci commented Apr 6, 2018

Build failed
Swift Test OS X Platform
Git Sha - 542fdab

@aschwaighofer
Copy link
Contributor Author

@shahmishal Howdy, pretty please :bowtie:

@shahmishal
Copy link
Member

@swift-ci please test

@swift-ci
Copy link
Contributor

swift-ci commented Apr 6, 2018

Build failed
Swift Test Linux Platform
Git Sha - 542fdab

@swift-ci
Copy link
Contributor

swift-ci commented Apr 6, 2018

Build failed
Swift Test OS X Platform
Git Sha - 542fdab

@shahmishal
Copy link
Member

@swift-ci please test

@swift-ci
Copy link
Contributor

swift-ci commented Apr 6, 2018

Build failed
Swift Test Linux Platform
Git Sha - 542fdab

@swift-ci
Copy link
Contributor

swift-ci commented Apr 6, 2018

Build failed
Swift Test OS X Platform
Git Sha - 542fdab

@aschwaighofer
Copy link
Contributor Author

@swift-ci Please smoke test macos

@aschwaighofer
Copy link
Contributor Author

@swift-ci Please smoke test

@aschwaighofer
Copy link
Contributor Author

The linux error is certainly unrelated. Merging.

Foundation/NSString.swift:309:25: error: value of type '_StringGuts' has no member '_isContiguousUTF16'

@aschwaighofer aschwaighofer merged commit baee4ab into swiftlang:swift-4.2-branch-03-26-2018 Apr 6, 2018
@aschwaighofer
Copy link
Contributor Author

@shahmishal Thanks for setting up the bot!

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