From 73349dc5ac33e904a1311829f2e42d923309e744 Mon Sep 17 00:00:00 2001 From: Redirect 'Slash' NIL Date: Sat, 17 Oct 2009 17:47:52 -0700 Subject: More MinGW C99 printf compliance Passing "--std=gun99" is unfortunately not sufficient to make current MinGW compilers conform with respect to checking printf format strings. (The C runtime seems not to have problems.) Fix by using a "gnu_printf" format specifier not "printf". --- src/helper/log.h | 12 ++++++++++-- 1 file changed, 10 insertions(+), 2 deletions(-) (limited to 'src/helper/log.h') diff --git a/src/helper/log.h b/src/helper/log.h index 7fc5a886..8f6ac770 100644 --- a/src/helper/log.h +++ b/src/helper/log.h @@ -28,6 +28,14 @@ #include "command.h" +/* To achieve C99 printf compatibility in MinGW, gnu_printf should */ +/* be used for __attribute__((format( ... ))) */ +#ifdef IS_MINGW +#define PRINTF_ATTRIBUTE_FORMAT gnu_printf +#else +#define PRINTF_ATTRIBUTE_FORMAT printf +#endif + /* logging priorities * LOG_LVL_SILENT - turn off all output. In lieu of try + catch this can be used as a * feeble ersatz. @@ -52,10 +60,10 @@ enum log_levels extern void log_printf(enum log_levels level, const char *file, int line, const char *function, const char *format, ...) -__attribute__ ((format (printf, 5, 6))); +__attribute__ ((format (PRINTF_ATTRIBUTE_FORMAT, 5, 6))); extern void log_printf_lf(enum log_levels level, const char *file, int line, const char *function, const char *format, ...) -__attribute__ ((format (printf, 5, 6))); +__attribute__ ((format (PRINTF_ATTRIBUTE_FORMAT, 5, 6))); extern int log_register_commands(struct command_context_s *cmd_ctx); extern int log_init(struct command_context_s *cmd_ctx); extern int set_log_output(struct command_context_s *cmd_ctx, FILE *output); -- cgit v1.2.3