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.

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

Add configure check for which printf length modifiers are available #325

Open
Snaipe opened this issue Jan 1, 2020 · 0 comments
Open

Add configure check for which printf length modifiers are available #325

Snaipe opened this issue Jan 1, 2020 · 0 comments

Comments

@Snaipe
Copy link
Owner

@Snaipe Snaipe commented Jan 1, 2020

On the topic of shaving preprocessor branches, I think the simplest thing to do here, since we only expose these format specifiers through cr_asprintf which in turn uses whatever vsnprintf implementation is available, is to add a configure-time check for the "ll" and "z" length modifiers, as well as what PRI{u,d}64 means on the current platform. We can then define the right format specifiers in a config.h installed in criterion/.

I don't think I'll do that in this PR, but I'll open an issue to remember to do that later.

Originally posted by @Snaipe in #324 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant
You can’t perform that action at this time.