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
[4.2] NSDecimalNumber: Add missing init(value:) initialisers. #1896
[4.2] NSDecimalNumber: Add missing init(value:) initialisers. #1896
Conversation
@swift-ci test 4.2 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
need to double check if we can add API here.
If I recall correctly I think these initialisers were missing originally but I can always remove them if necessary. The |
@spevans yes, I think we should try to not add API (in this case the inits) unless strictly necessary. This might break code (because it might cause ambiguity). |
- Set the mantissa correctly. (cherry picked from commit 641e2cb)
9efe369
to
86c4684
Compare
@swift-ci test 4.2 |
1 similar comment
@swift-ci test 4.2 |
@parkera - is there an ability to run a source compatibility test like on swift with Foundation? |
@weissi I removed the other |
@compnerd not sure about the source compatibility test; @shahmishal do you know if we can trigger that from here? |
|
correctly.
(cherry picked from commit 641e2cb)