Skip to content
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

Fixed SR-5971 #1470

Closed
wants to merge 2 commits into from
Closed

Fixed SR-5971 #1470

wants to merge 2 commits into from

Conversation

notorca
Copy link
Contributor

@notorca notorca commented Mar 11, 2018

CFCharacterSetInitWithCharactersInString crashed on strings with surrogate characters. Now _CFCharacterSetInitWithCharactersInString is semantically equivalent to CFCharacterSetInitWithCharactersInString from core foundation.

CharacterSet is crashes when created from some strings
_CFCharacterSetInitWithCharactersInString now equivalent to CFCharacterSetCreateWithCharactersInString
@spevans
Copy link
Collaborator

spevans commented Mar 11, 2018

@swift-ci please test

@spevans
Copy link
Collaborator

spevans commented Mar 11, 2018

@notorca Could you provide a PR against the master branch? That branch is always patched first before PRs are optionally applied to other branches. Thanks!

@notorca
Copy link
Contributor Author

notorca commented Mar 11, 2018

Closed in favour of #1471

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.

None yet

2 participants