Skip to content

Conversation

@Kaiede
Copy link
Contributor

@Kaiede Kaiede commented Dec 7, 2018

See PR #1703 for the original details around the fix.

Work for 4.2 originally regressed this behavior from 4.1 and earlier, so this really should be fixed in 4.2 for Linux as well as the original commit to master.

The original change for this did not check the behavior.

A better implementation here is to use the one from Swift itself, which also matches the initializer that this extension also adds.
@Kaiede
Copy link
Contributor Author

Kaiede commented Dec 7, 2018

Paging @millenomi and @parkera, who suggested this should also get pulled to the 4.2 branch.

@parkera
Copy link
Contributor

parkera commented Dec 11, 2018

I'm going to merge this so it's staged for a potential release.

@parkera
Copy link
Contributor

parkera commented Dec 11, 2018

@swift-ci test and merge

@swift-ci swift-ci merged commit af89a49 into swiftlang:swift-4.2-branch Dec 11, 2018
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