diff options
| author | Alejandro Colomar <alx.manpages@gmail.com> | 2022-03-11 01:59:24 +0100 |
|---|---|---|
| committer | Alejandro Colomar <alx.manpages@gmail.com> | 2022-04-27 01:14:22 +0200 |
| commit | 0b79735b503cc0a35062799a8ac45f169f0af0f7 (patch) | |
| tree | b2cc62f7135dd8facd060fbcbab6fd78ba7de7d8 /src/nxt_clang.h | |
| parent | a3d19f71a205d31ce141dcfd8880f7ed13bd65e5 (diff) | |
| download | unit-0b79735b503cc0a35062799a8ac45f169f0af0f7.tar.gz unit-0b79735b503cc0a35062799a8ac45f169f0af0f7.tar.bz2 | |
Added NXT_MAYBE_UNUSED for __attribute__((__unused__)).
When testing some configurations of compilers and OSes, I noticed
that clang(1) 13 on Debian caused a function to be compiled but
unused, and the compiler triggered a compile error.
To avoid that error, use __attribute__((__unused__)). Let's call
our wrapper NXT_MAYBE_UNUSED, since it describes itself more
precisely than the GCC attribute name. It's also the name that
C2x (likely C23) has given to the standard attribute, which is
[[maybe_unused]], so it's also likely to be more readable because
of that name being in ISO C.
Diffstat (limited to 'src/nxt_clang.h')
| -rw-r--r-- | src/nxt_clang.h | 11 |
1 files changed, 11 insertions, 0 deletions
diff --git a/src/nxt_clang.h b/src/nxt_clang.h index a10de08a..26afba7a 100644 --- a/src/nxt_clang.h +++ b/src/nxt_clang.h @@ -132,6 +132,17 @@ nxt_prefetch(a) #endif +#if (NXT_HAVE_GCC_ATTRIBUTE_UNUSED) + +#define NXT_MAYBE_UNUSED __attribute__((__unused__)) + +#else + +#define NXT_MAYBE_UNUSED + +#endif + + #if (NXT_HAVE_BUILTIN_POPCOUNT) #define nxt_popcount __builtin_popcount |