Toolchain/binutils-gdb
1
0
Fork 0
forked from Imagelibrary/binutils-gdb
Code Pull Requests Activity

libctf: use __attribute__((__gnu_printf__)) where appropriate

Browse Source 
We don't use any GNU-specific printf args, but this prevents warnings about
%z, observed on MinGW even though every libc anyone is likely to use there
supports %z perfectly well, and we're not stopping using it just because
MinGW complains.  Doing this means we stand more chance of seeing *actual*
problems on such platforms without them being drowned in noise.

We turn this off on clang, which doesn't support __gnu_printf__.

Suggested by Eli Zaretskii.

libctf/
	PR libctf/31863
	* ctf-impl.h (_libctf_printflike_): Use __gnu_printf__.
This commit is contained in:
Nick Alcock
2025-06-03 13:39:33 +01:00
parent fe282c7bb5
commit 0d366df443
1 changed files with 5 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
libctf/ctf-impl.h
View File

@@ -67,8 +67,13 @@ extern "C"
macros glibc may introduce, which have names of the pattern macros glibc may introduce, which have names of the pattern
__attribute_blah__. */ __attribute_blah__. */
#if defined (__clang__)
#define _libctf_printflike_(string_index,first_to_check) \ #define _libctf_printflike_(string_index,first_to_check) \
__attribute__ ((__format__ (__printf__, (string_index), (first_to_check)))) __attribute__ ((__format__ (__printf__, (string_index), (first_to_check))))
#else
#define _libctf_printflike_(string_index,first_to_check) \
__attribute__ ((__format__ (__gnu_printf__, (string_index), (first_to_check))))
#endif
#define _libctf_unlikely_(x) __builtin_expect ((x), 0) #define _libctf_unlikely_(x) __builtin_expect ((x), 0)
#define _libctf_unused_ __attribute__ ((__unused__)) #define _libctf_unused_ __attribute__ ((__unused__))
#define _libctf_malloc_ __attribute__((__malloc__)) #define _libctf_malloc_ __attribute__((__malloc__))