[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]

Re: [pygame] Patch for win-amd64





On 7/27/2011 3:51 PM, Lenard Lindstrom wrote:
Hi René,

No, you have beaten me to this one. I have no access to a 64bit machine
for testing. I had a quick look at the patch and it seems sound. I would
check the "n" format in return Py_BuildValue("(nn)", self->stride0,
self->stride1) though. I don't know when it was introduced to Python.

Lenard

Hi,

It was introduced in Python 2.5:

<http://docs.python.org/c-api/arg.html#Py_BuildValue>

"""
n (int) [Py_ssize_t]

    Convert a C Py_ssize_t to a Python integer or long integer.

    New in version 2.5.
"""

Christoph



On 27/07/11 04:45 AM, René Dudfield wrote:
Hi,

thanks. Looks good to me. You've changed some parts to Py_ssize_t and
c_ssize_t, from the 32bit types.

Do all the tests pass on 32bit as well?

I'll try it out later tonight, and apply (unless Lenard beats me to it).

cheers!


On Tue, Jul 26, 2011 at 3:17 AM, Christoph Gohlke <cgohlke@xxxxxxx
<mailto:cgohlke@xxxxxxx>> wrote:

Hello,

please consider the attached patch. It fixes some array interface
related crashes and test failures on the win-amd64 platform.

Christoph