diff options
author | Thomas Heller <theller@ctypes.org> | 2008-09-29 20:03:53 (GMT) |
---|---|---|
committer | Thomas Heller <theller@ctypes.org> | 2008-09-29 20:03:53 (GMT) |
commit | 28acc6cae585d3cae81755f52fed66a4cb04c128 (patch) | |
tree | 3dcbf099ad06bf8b5bbd8be69e8ba901dba7a4f5 | |
parent | 4970215bb544977f958c189ffcc440c0647d52b8 (diff) | |
download | cpython-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.py | 5 | ||||
-rw-r--r-- | Modules/_ctypes/cfield.c | 8 |
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 */ |