msi/tests: Ok_() takes printf-style arguments.

This lets the compiler check that the format and argument sizes match.

msi/tests: Ok_() takes printf-style arguments.
a80b183a · Francois Gouget · Alexandre Julliard · b6fb1b89 · a80b183a
Commit a80b183a authored Apr 21, 2023 by Francois Gouget Committed by Alexandre Julliard Apr 27, 2023
Show whitespace changes
Inline Side-by-side

Showing with 7 additions and 1 deletion

custom.c dlls/msi/tests/custom.c +7 -1

No files found.
--- a/dlls/msi/tests/custom.c
+++ b/dlls/msi/tests/custom.c
@@ -32,9 +32,15 @@
 #include <msiquery.h>
 #include <msidefs.h>

+#if defined(__MINGW32__) || (!defined(__WINE_USE_MSVCRT) && (defined(__GNUC__) || defined(__clang__)))
+#define __WINE_PRINTF_ATTR(fmt,args) __attribute__((format (printf,fmt,args)))
+#else
+#define __WINE_PRINTF_ATTR(fmt,args)
+#endif
+
 static int todo_level, todo_do_loop;

-static void WINAPIV ok_(MSIHANDLE hinst, int todo, const char *file, int line, int condition, const char *msg, ...)
+static void WINAPIV  __WINE_PRINTF_ATTR(6,7) ok_(MSIHANDLE hinst, int todo, const char *file, int line, int condition, const char *msg, ...)
 {
    static char buffer[2000];
    MSIHANDLE record;