Author steve.dower
Recipients amaury.forgeotdarc, belopolsky, meador.inge, steve.dower
Date 2014-10-26.20:32:05
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1414355526.1.0.122133649749.issue22732@psf.upfronthosting.co.za>
In-reply-to
Content
The test_pass_pointer and test_int_pointer_arg tests are inconsistent between 32-bit and 64-bit builds because c_long is always 32 bits but the function being called may return a 64-bit value (it's a pointer, so c_long may truncate it).

I'd prefer to have a c_intptr type to use, but this patch uses c_longlong in the test if that's the size of a pointer.

Just looking for a quick review so I can check this into default. Not sure why it doesn't repro with VC10, but it certainly does with later compilers.
History
Date User Action Args
2014-10-26 20:32:06steve.dowersetrecipients: + steve.dower, amaury.forgeotdarc, belopolsky, meador.inge
2014-10-26 20:32:06steve.dowersetmessageid: <1414355526.1.0.122133649749.issue22732@psf.upfronthosting.co.za>
2014-10-26 20:32:06steve.dowerlinkissue22732 messages
2014-10-26 20:32:05steve.dowercreate