Skip to content

[stdlib] UnicodeScalarProperties: use RawValue over UInt32 #18643

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

Merged
merged 2 commits into from
Aug 12, 2018

Conversation

troughton
Copy link
Contributor

On Windows, __swift_stdlib_UNumericType and __swift_stdlib_UCharCategory are imported as Int32s rather than UInt32. Change the constructors to use the type's inferred RawValue rather than always UInt32.

On Windows, `__swift_stdlib_UNumericType` and `__swift_stdlib_UCharCategory` are imported as `Int32`s rather than `UInt32`. Change the constructors to use the type's inferred `RawValue` rather than always `UInt32`.
@troughton
Copy link
Contributor Author

CC @milseman

@milseman
Copy link
Member

@swift-ci please test

Copy link
Member

@milseman milseman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@milseman
Copy link
Member

(first time resolving a merge conflict through Github browser, let's see how it goes)

@swift-ci please test

@milseman
Copy link
Member

@swift-ci please test

@swift-ci
Copy link
Contributor

Build failed
Swift Test OS X Platform
Git Sha - 839d92b

@swift-ci
Copy link
Contributor

Build failed
Swift Test Linux Platform
Git Sha - 839d92b

@milseman milseman merged commit 97cb87e into swiftlang:master Aug 12, 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