[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH 10/11] osdep: Remove local definition of macro offse
From: |
Stefan Hajnoczi |
Subject: |
[Qemu-devel] [PATCH 10/11] osdep: Remove local definition of macro offsetof |
Date: |
Thu, 8 Mar 2012 15:37:20 +0000 |
From: Stefan Weil <address@hidden>
The macro offsetof is defined in stddef.h. It is conforming to
the standards C89, C99 and POSIX.1-2001 (see man page), so it
is a sufficiently old standard.
Therefore chances are very high that QEMU never needs a local
definition of this macro.
osdep.h already includes stddef.h, so this patch simply removes
the unneeded code from the files configure and osdep.h.
If we ever need the local definition again, it should be added
to compiler.h (the macro is usually provided with the compiler,
it is not OS specific).
Signed-off-by: Stefan Weil <address@hidden>
Signed-off-by: Stefan Hajnoczi <address@hidden>
---
configure | 14 --------------
osdep.h | 3 ---
2 files changed, 0 insertions(+), 17 deletions(-)
diff --git a/configure b/configure
index e19b6e3..ed3d907 100755
--- a/configure
+++ b/configure
@@ -2524,17 +2524,6 @@ if test "$darwin" != "yes" -a "$mingw32" != "yes" -a
"$solaris" != yes -a \
fi
##########################################
-# check if the compiler defines offsetof
-
-need_offsetof=yes
-cat > $TMPC << EOF
-#include <stddef.h>
-int main(void) { struct s { int f; }; return offsetof(struct s, f); }
-EOF
-if compile_prog "" "" ; then
- need_offsetof=no
-fi
-
# spice probe
if test "$spice" != "no" ; then
cat > $TMPC << EOF
@@ -3199,9 +3188,6 @@ fi
if test "$tcg_interpreter" = "yes" ; then
echo "CONFIG_TCG_INTERPRETER=y" >> $config_host_mak
fi
-if test "$need_offsetof" = "yes" ; then
- echo "CONFIG_NEED_OFFSETOF=y" >> $config_host_mak
-fi
if test "$fdatasync" = "yes" ; then
echo "CONFIG_FDATASYNC=y" >> $config_host_mak
fi
diff --git a/osdep.h b/osdep.h
index 432b91e..0350383 100644
--- a/osdep.h
+++ b/osdep.h
@@ -26,9 +26,6 @@
#define unlikely(x) __builtin_expect(!!(x), 0)
#endif
-#ifdef CONFIG_NEED_OFFSETOF
-#define offsetof(TYPE, MEMBER) ((size_t) &((TYPE *) 0)->MEMBER)
-#endif
#ifndef container_of
#define container_of(ptr, type, member) ({ \
const typeof(((type *) 0)->member) *__mptr = (ptr); \
--
1.7.9.1
- [Qemu-devel] [PULL v2 00/11] Trivial patches for 25 February to 7 March 2012, Stefan Hajnoczi, 2012/03/08
- [Qemu-devel] [PATCH 06/11] libcacard: Use format specifier %u instead of %d for unsigned values, Stefan Hajnoczi, 2012/03/08
- [Qemu-devel] [PATCH 08/11] Spelling fixes in comments (it's -> its), Stefan Hajnoczi, 2012/03/08
- [Qemu-devel] [PATCH 01/11] sh7750: Remove redundant 'struct' from MemoryRegionOps, Stefan Hajnoczi, 2012/03/08
- [Qemu-devel] [PATCH 05/11] Fix sign of sscanf format specifiers, Stefan Hajnoczi, 2012/03/08
- [Qemu-devel] [PATCH 09/11] libcacard: Spelling and grammar fixes in documentation, Stefan Hajnoczi, 2012/03/08
- [Qemu-devel] [PATCH 04/11] block/vmdk: Fix warning from splint (comparision of unsigned value), Stefan Hajnoczi, 2012/03/08
- [Qemu-devel] [PATCH 02/11] qom: Fix spelling in documentation, Stefan Hajnoczi, 2012/03/08
- [Qemu-devel] [PATCH 03/11] qmp: Fix spelling fourty -> forty, Stefan Hajnoczi, 2012/03/08
- [Qemu-devel] [PATCH 10/11] osdep: Remove local definition of macro offsetof,
Stefan Hajnoczi <=
- [Qemu-devel] [PATCH 07/11] vnc: Add break statement, Stefan Hajnoczi, 2012/03/08
- [Qemu-devel] [PATCH 11/11] configure: Quote the configure args printed in config.log, Stefan Hajnoczi, 2012/03/08
- Re: [Qemu-devel] [PULL v2 00/11] Trivial patches for 25 February to 7 March 2012, Anthony Liguori, 2012/03/09