forked from mpaland/printf
-
Notifications
You must be signed in to change notification settings - Fork 62
Open
Labels
resolved-on-developA changeset fixing this issue has been commiutted to the development branchA changeset fixing this issue has been commiutted to the development branchtask
Description
If we don't define PRINTF_INCLUDE_CONFIG_H, we may get the warning:
In file included from /home/eyalroz/src/mine/printf/test/aliasing.c:2:
/home/eyalroz/src/mine/printf/src/printf/printf.h:42:5: warning: "PRINTF_INCLUDE_CONFIG_H" is not defined, evaluates to 0 [-Wundef]
42 | #if PRINTF_INCLUDE_CONFIG_H
| ^~~~~~~~~~~~~~~~~~~~~~~
That warning is... arguably invalid, because it's well-defined in the standard that undefined -> evaluates to 0, but - it is what it is.
So, let's replace this check with:
#if defined(PRINTF_INCLUDE_CONFIG_H) && PRINTF_INCLUDE_CONFIG_H
Metadata
Metadata
Assignees
Labels
resolved-on-developA changeset fixing this issue has been commiutted to the development branchA changeset fixing this issue has been commiutted to the development branchtask