[Python-checkins] r66683 - in python/trunk: Lib/ctypes/test/test_bitfields.py Modules/_ctypes/cfield.c
thomas.heller
python-checkins at python.org
Mon Sep 29 21:56:24 CEST 2008
Author: thomas.heller
Date: Mon Sep 29 21:56:24 2008
New Revision: 66683
Log:
Fix issue #3547 for MingW, update comments.
Modified:
python/trunk/Lib/ctypes/test/test_bitfields.py
python/trunk/Modules/_ctypes/cfield.c
Modified: python/trunk/Lib/ctypes/test/test_bitfields.py
==============================================================================
--- python/trunk/Lib/ctypes/test/test_bitfields.py (original)
+++ python/trunk/Lib/ctypes/test/test_bitfields.py Mon Sep 29 21:56:24 2008
@@ -223,8 +223,9 @@
("d", c_short, 4),
("e", c_short, 4),
("f", c_int, 24)]
- # MS compilers do NOT combine c_short and c_int into
- # one field, gcc does.
+ # MSVC does NOT combine c_short and c_int into one field, GCC
+ # does (unless GCC is run with '-mms-bitfields' which
+ # produces code compatible with MSVC).
if os.name in ("nt", "ce"):
self.failUnlessEqual(sizeof(X), sizeof(c_int) * 4)
else:
Modified: python/trunk/Modules/_ctypes/cfield.c
==============================================================================
--- python/trunk/Modules/_ctypes/cfield.c (original)
+++ python/trunk/Modules/_ctypes/cfield.c Mon Sep 29 21:56:24 2008
@@ -65,10 +65,12 @@
}
if (bitsize /* this is a bitfield request */
&& *pfield_size /* we have a bitfield open */
-#if defined(MS_WIN32) && !defined(__MINGW32__)
- && dict->size * 8 == *pfield_size /* MSVC */
+#ifdef MS_WIN32
+ /* MSVC, GCC with -mms-bitfields */
+ && dict->size * 8 == *pfield_size
#else
- && dict->size * 8 <= *pfield_size /* GCC, MINGW */
+ /* GCC */
+ && dict->size * 8 <= *pfield_size
#endif
&& (*pbitofs + bitsize) <= *pfield_size) {
/* continue bit field */
More information about the Python-checkins
mailing list