summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorThomas Heller <theller@ctypes.org>2008-09-29 20:03:53 (GMT)
committerThomas Heller <theller@ctypes.org>2008-09-29 20:03:53 (GMT)
commit28acc6cae585d3cae81755f52fed66a4cb04c128 (patch)
tree3dcbf099ad06bf8b5bbd8be69e8ba901dba7a4f5
parent4970215bb544977f958c189ffcc440c0647d52b8 (diff)
downloadcpython-28acc6cae585d3cae81755f52fed66a4cb04c128.zip
cpython-28acc6cae585d3cae81755f52fed66a4cb04c128.tar.gz
cpython-28acc6cae585d3cae81755f52fed66a4cb04c128.tar.bz2
Merged revisions 66683 via svnmerge from
svn+ssh://pythondev@svn.python.org/python/trunk ........ r66683 | thomas.heller | 2008-09-29 21:56:24 +0200 (Mo, 29 Sep 2008) | 1 line Fix issue #3547 for MingW, update comments. ........
-rw-r--r--Lib/ctypes/test/test_bitfields.py5
-rw-r--r--Modules/_ctypes/cfield.c8
2 files changed, 8 insertions, 5 deletions
diff --git a/Lib/ctypes/test/test_bitfields.py b/Lib/ctypes/test/test_bitfields.py
index 06c4023..de1c619 100644
--- a/Lib/ctypes/test/test_bitfields.py
+++ b/Lib/ctypes/test/test_bitfields.py
@@ -223,8 +223,9 @@ class BitFieldTest(unittest.TestCase):
("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:
diff --git a/Modules/_ctypes/cfield.c b/Modules/_ctypes/cfield.c
index 9040dbb..b6f41fe 100644
--- a/Modules/_ctypes/cfield.c
+++ b/Modules/_ctypes/cfield.c
@@ -61,10 +61,12 @@ CField_FromDesc(PyObject *desc, Py_ssize_t index,
}
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 */