-
Notifications
You must be signed in to change notification settings - Fork 471
Fixed printf format macro for intptr_t arguments. #448
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
Conversation
@swift-ci please test |
1 similar comment
@swift-ci please test |
It seems that the test failed due to the following error, which seems unrelated to this change:
Is there anything I can do to make this pass? |
@swift-ci please test |
This time a number of tests in Foundation failed, which again seem completely unrelated to this change. @compnerd any idea? Thanks! |
mmm...that does go through libdispatch, so it could be a difference, if you can find the failure in the logs, it would be helpful. |
There seem to be a couple of errors / failures, none of which seem to me to be related to my changes in First, there is this error in "Performing configure step for 'compiler-rt'":
And then a number of tests fail: TestDateIntervalFormatter
TestNSNumber.test_descriptionWithLocale
TestScanner.testScanXXX
In the end it looks like these cause "TestFoundation.xctest" to fail:
|
The compiler-rt thing is not important. The TestFoundation failure is however. I didn't manage to spot the test that failed. It doesn't seem like it failed due to a segfault though. |
These are the tests that failed:
I also saw that #451 has the same tests failing, which seems like further indication that this a different issue with the tests. |
This sounds like an issue with ICU. |
@swift-ci please test |
1 similar comment
@swift-ci please test |
dsema_value is of type long, and dsema_orig of type intptr_t.
I’ve updated the patch to be based on the latest changes after the merge. |
@swift-ci please test |
Fixed printf format macro for intptr_t arguments. Signed-off-by: Kim Topley <ktopley@apple.com>
Was using PRId64 instead of PRIdPTR, which would be incorrect when compiling for 32-bit.