Skip to content
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.

Sign up for GitHub

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

Jump to bottom

No test coverage of the alternative implementation with %f, %g, %e, %a #111

Open
eyalroz opened this issue Jul 21, 2021 · 0 comments
Open

No test coverage of the alternative implementation with %f, %g, %e, %a #111

eyalroz opened this issue Jul 21, 2021 · 0 comments

Comments

@eyalroz
Copy link

eyalroz commented Jul 21, 2021

The alternative implementation flag ('#') can be used with the specifiers %f, %g, %e, %a, %o and %x (and their uppercase versions). However, test_suite.cpp only uses the flag with %o and %x (and %d, %b, %iand%u` if non-standard format strings are supported).
KarlK90 pushed a commit to qmk/printf that referenced this issue Jul 7, 2022
@eyalroz
Fixes mpaland#111, mpaland#112: test suite now includes `printf_confi…
df0fb9e 
…g.h` before `printf.c`; and separated cases of what's expected to be printed based on the value of `PRINTF_MAX_INTEGRAL_DIGITS_FOR_DECIMAL`.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@eyalroz