Hi Lisandro, Lisandro Dalcin wrote: > The only thing that I'm not completely sure about is the > implementation of this, though it works fine in my 32bit Linux box > > static INLINE size_t __pyx_PyInt_AsSize_t(PyObject* b);
I'd need to see the implementation to comment on this. > Of course, this change is not backward compatible... Cython will not > compile the code: > > cdef extern from "stdlib.h": > ctypedef unsigned long size_t Is this because of an explicit error, or just a side effect of the implementation? > What should be the behavior if the user tries to ctypedef 'Py_ssize_t' > or 'size_t', provided that Cython treats them as pre-defined C types? I didn't care at the time when I implemented the Py_ssize_t stuff, but when you ask now, I prefer getting at least a warning. I doubt that Cython handles a "ctypedef ... int" gracefully, for example. > Finally, can I push the changes? Could you file a bug report for now (if only for documentation purposes) and post the patch there? We should be a bit careful with major changes by now, as we are currently trying to stabilise Cython for the 0.11 release. Thanks, Stefan _______________________________________________ Cython-dev mailing list [email protected] http://codespeak.net/mailman/listinfo/cython-dev
