[pywin32-bugs] [ pywin32-Bugs-1709340 ] ByRef 2D arrays of 1 Byte improperly handled
OLD project page for the Python extensions for Windows
Brought to you by:
mhammond
From: SourceForge.net <no...@so...> - 2007-05-12 12:14:51
|
Bugs item #1709340, was opened at 2007-04-28 13:26 Message generated for change (Comment added) made by wyrock You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=551954&aid=1709340&group_id=78018 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: win32 Group: None Status: Open Resolution: None Priority: 5 Private: No Submitted By: wyrock (wyrock) Assigned to: Mark Hammond (mhammond) Summary: ByRef 2D arrays of 1 Byte improperly handled Initial Comment: If a COM module passes a 2D array where one each dimension is only one byte, then win32com only returns a buffer. The buffer prevents access from the other dimension of the array. It is expected to get an array of buffers. The bug lies in oleargs.cpp. I've attached an updated version of oleargs.cpp and also some modified versions of the com_testing code to show the problem. Could someone please review the proposed fix and commit the change? Thanks, ---------------------------------------------------------------------- >Comment By: wyrock (wyrock) Date: 2007-05-12 07:14 Message: Logged In: YES user_id=1681725 Originator: YES File Added: test2dByRef.py ---------------------------------------------------------------------- Comment By: wyrock (wyrock) Date: 2007-05-12 07:13 Message: Logged In: YES user_id=1681725 Originator: YES Attaching the cvs diff with the fix (in oleargs.cpp) and some modifications to the test bench that test for the issue. File Added: diff.out ---------------------------------------------------------------------- Comment By: Mark Hammond (mhammond) Date: 2007-05-06 06:00 Message: Logged In: YES user_id=14198 Originator: NO Thanks! I haven't looked yet, but I can believe what you say is true. I'll (try to) get to this before the next release (but a .diff against the test suite instead of the .zip with a complete copy would help even more!) If you have the c compiler handy, then 'setup.py install' from CVS is quite an easy way to get a build from sources (but I digress... :) ---------------------------------------------------------------------- Comment By: wyrock (wyrock) Date: 2007-05-05 17:10 Message: Logged In: YES user_id=1681725 Originator: YES File Added: oleargs.new.cpp ---------------------------------------------------------------------- Comment By: wyrock (wyrock) Date: 2007-04-28 13:35 Message: Logged In: YES user_id=1681725 Originator: YES File Added: testSmaller.py ---------------------------------------------------------------------- Comment By: wyrock (wyrock) Date: 2007-04-28 13:34 Message: Logged In: YES user_id=1681725 Originator: YES File Added: com_testing2.zip ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=551954&aid=1709340&group_id=78018 |