[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH 05/12] compiler.h: drop __printf__ macro MinGW/glib workaround
From: |
marcandre . lureau |
Subject: |
[PATCH 05/12] compiler.h: drop __printf__ macro MinGW/glib workaround |
Date: |
Thu, 24 Feb 2022 22:36:54 +0400 |
From: Marc-André Lureau <marcandre.lureau@redhat.com>
This workaround was added in commit 95df51a4 ("w32: Always use standard
instead of native format strings"), as it claimed glib was using
__printf__ attribute. This is surprising, since glib has always used
G_GNUC_PRINTF which, as the name implies, uses __gnu_printf__ when
possible.
Apparently, the workaound is no longer relevant though, I don't see
the warnings.
Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>
---
include/qemu/compiler.h | 8 --------
1 file changed, 8 deletions(-)
diff --git a/include/qemu/compiler.h b/include/qemu/compiler.h
index 2704c314dcac..eb29b72c14d7 100644
--- a/include/qemu/compiler.h
+++ b/include/qemu/compiler.h
@@ -73,14 +73,6 @@
#define QEMU_BUILD_BUG_ON_ZERO(x) (sizeof(QEMU_BUILD_BUG_ON_STRUCT(x)) - \
sizeof(QEMU_BUILD_BUG_ON_STRUCT(x)))
-#if !defined(__clang__) && defined(_WIN32)
-/*
- * Map __printf__ to __gnu_printf__ because we want standard format strings
even
- * when MinGW or GLib include files use __printf__.
- */
-# define __printf__ __gnu_printf__
-#endif
-
#ifndef __has_warning
#define __has_warning(x) 0 /* compatibility with non-clang compilers */
#endif
--
2.35.1.273.ge6ebfd0e8cbb
- [PATCH 01/12] compiler.h: replace QEMU_WARN_UNUSED_RESULT with G_GNUC_WARN_UNUSED_RESULT, (continued)
[PATCH 06/12] Replace config-time define HOST_WORDS_BIGENDIAN, marcandre . lureau, 2022/02/24
[PATCH 09/12] scripts/modinfo-collect: remove unused/dead code, marcandre . lureau, 2022/02/24
[PATCH 08/12] Move HOST_LONG_BITS to compiler.h, marcandre . lureau, 2022/02/24