[Cython] ready to push changes to make Cython understand 'size_t' as a native C type

[Stefan Behnel]

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