Skip to content

[4.2] SR-1464: NSNumber.description is not compatible between OS X and linux #1973

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
Mar 6, 2019

Conversation

spevans
Copy link
Contributor

@spevans spevans commented Mar 6, 2019

  • Use Swift's stringification of numbers for .description and .stringValue

  • Use String(format:locale:...) with the correct format for the
    NSNumber type if locale is not nil.

  • Enable use of __CFStringFormatLocalizedNumber() on
    DEPLOYMENT_TARGET_LINUX.

(cherry picked from commit 6450221)

  • SR-1464: Add NSDecimalNumber.description

(cherry picked from commit 7cb608f)

spevans added 2 commits March 6, 2019 16:34
- Use Swift's stringification of numbers for .description and .stringValue

- Use String(format:locale:...) with the correct format for the
  NSNumber type if locale is not nil.

- Enable use of __CFStringFormatLocalizedNumber() on
  DEPLOYMENT_TARGET_LINUX.

(cherry picked from commit 6450221)
@spevans
Copy link
Contributor Author

spevans commented Mar 6, 2019

@swift-ci test 4.2

@spevans
Copy link
Contributor Author

spevans commented Mar 6, 2019

New version of #1929 which should work on ARM alongside #1972

@spevans
Copy link
Contributor Author

spevans commented Mar 6, 2019

cc @weissi @futurejones

@spevans spevans merged commit 684bb66 into swiftlang:swift-4.2-branch Mar 6, 2019
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.

2 participants