-
Notifications
You must be signed in to change notification settings - Fork 2k
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
tests_printf_float: fixed precision in test case #5383
Conversation
Looks valid but I don't see why adding only one decimal... (or two or three...) This is only for the test? |
Same here, why not check for the full value? |
The test was designed like this at the beginning, I'm surprised that with the wrong format string. Checking the full value, or 1 or 2 digits is a mater of taste. |
already proposed a more complete version of the test: #5384 |
@aabadie also see #5512 (comment) |
While testing for Release 2016.04 it turned out that the test always failed. A precision value according to the reference string was missing.