[Python-checkins] r88284 - in python/branches/py3k: Lib/ctypes/test/test_callbacks.py Misc/ACKS Misc/NEWS Modules/_ctypes/_ctypes_test.c Modules/_ctypes/libffi_msvc/ffi.c
antoine.pitrou
python-checkins at python.org
Mon Jan 31 22:08:58 CET 2011
Author: antoine.pitrou
Date: Mon Jan 31 22:08:57 2011
New Revision: 88284
Log:
Issue #8275: Fix passing of callback arguments with ctypes under Win64.
Patch by Stan Mihai. Ok'ed by Georg.
Modified:
python/branches/py3k/Lib/ctypes/test/test_callbacks.py
python/branches/py3k/Misc/ACKS
python/branches/py3k/Misc/NEWS
python/branches/py3k/Modules/_ctypes/_ctypes_test.c
python/branches/py3k/Modules/_ctypes/libffi_msvc/ffi.c
Modified: python/branches/py3k/Lib/ctypes/test/test_callbacks.py
==============================================================================
--- python/branches/py3k/Lib/ctypes/test/test_callbacks.py (original)
+++ python/branches/py3k/Lib/ctypes/test/test_callbacks.py Mon Jan 31 22:08:57 2011
@@ -200,6 +200,42 @@
windll.user32.EnumWindows(EnumWindowsCallbackFunc, 0)
+ def test_callback_register_int(self):
+ # Issue #8275: buggy handling of callback args under Win64
+ # NOTE: should be run on release builds as well
+ dll = CDLL(_ctypes_test.__file__)
+ CALLBACK = CFUNCTYPE(c_int, c_int, c_int, c_int, c_int, c_int)
+ # All this function does is call the callback with its args squared
+ func = dll._testfunc_cbk_reg_int
+ func.argtypes = (c_int, c_int, c_int, c_int, c_int, CALLBACK)
+ func.restype = c_int
+
+ def callback(a, b, c, d, e):
+ return a + b + c + d + e
+
+ result = func(2, 3, 4, 5, 6, CALLBACK(callback))
+ self.assertEqual(result, callback(2*2, 3*3, 4*4, 5*5, 6*6))
+
+ def test_callback_register_double(self):
+ # Issue #8275: buggy handling of callback args under Win64
+ # NOTE: should be run on release builds as well
+ dll = CDLL(_ctypes_test.__file__)
+ CALLBACK = CFUNCTYPE(c_double, c_double, c_double, c_double,
+ c_double, c_double)
+ # All this function does is call the callback with its args squared
+ func = dll._testfunc_cbk_reg_double
+ func.argtypes = (c_double, c_double, c_double,
+ c_double, c_double, CALLBACK)
+ func.restype = c_double
+
+ def callback(a, b, c, d, e):
+ return a + b + c + d + e
+
+ result = func(1.1, 2.2, 3.3, 4.4, 5.5, CALLBACK(callback))
+ self.assertEqual(result,
+ callback(1.1*1.1, 2.2*2.2, 3.3*3.3, 4.4*4.4, 5.5*5.5))
+
+
################################################################
if __name__ == '__main__':
Modified: python/branches/py3k/Misc/ACKS
==============================================================================
--- python/branches/py3k/Misc/ACKS (original)
+++ python/branches/py3k/Misc/ACKS Mon Jan 31 22:08:57 2011
@@ -574,6 +574,7 @@
Mike Meyer
Steven Miale
Trent Mick
+Stan Mihai
Aristotelis Mikropoulos
Damien Miller
Chad Miller
Modified: python/branches/py3k/Misc/NEWS
==============================================================================
--- python/branches/py3k/Misc/NEWS (original)
+++ python/branches/py3k/Misc/NEWS Mon Jan 31 22:08:57 2011
@@ -13,6 +13,9 @@
Library
-------
+- Issue #8275: Fix passing of callback arguments with ctypes under Win64.
+ Patch by Stan Mihai.
+
What's New in Python 3.2 Release Candidate 2?
=============================================
Modified: python/branches/py3k/Modules/_ctypes/_ctypes_test.c
==============================================================================
--- python/branches/py3k/Modules/_ctypes/_ctypes_test.c (original)
+++ python/branches/py3k/Modules/_ctypes/_ctypes_test.c Mon Jan 31 22:08:57 2011
@@ -12,6 +12,20 @@
/* some functions handy for testing */
+EXPORT(int)
+_testfunc_cbk_reg_int(int a, int b, int c, int d, int e,
+ int (*func)(int, int, int, int, int))
+{
+ return func(a*a, b*b, c*c, d*d, e*e);
+}
+
+EXPORT(double)
+_testfunc_cbk_reg_double(double a, double b, double c, double d, double e,
+ double (*func)(double, double, double, double, double))
+{
+ return func(a*a, b*b, c*c, d*d, e*e);
+}
+
EXPORT(void)testfunc_array(int values[4])
{
printf("testfunc_array %d %d %d %d\n",
Modified: python/branches/py3k/Modules/_ctypes/libffi_msvc/ffi.c
==============================================================================
--- python/branches/py3k/Modules/_ctypes/libffi_msvc/ffi.c (original)
+++ python/branches/py3k/Modules/_ctypes/libffi_msvc/ffi.c Mon Jan 31 22:08:57 2011
@@ -380,7 +380,7 @@
short bytes;
char *tramp;
#ifdef _WIN64
- int mask;
+ int mask = 0;
#endif
FFI_ASSERT (cif->abi == FFI_SYSV);
More information about the Python-checkins
mailing list