Bruce Momjian <pgman@candle.pha.pa.us> writes:
> PyGreSQL causes a segfault when used with a Python executable that was
> compiled with --with-pymalloc. This change fixes that. Thanks to
> Dave Wallace <dwallace@udel.edu>
>
> ---------------------------------------------------------------------------
>
> The patch changes PyMem_DEL(self) to PyObject_DEL(self).
>
> This is with python 1.5. I suggest that we should reverse out this patch.
> The fix for --with-pymalloc clearly is causing problems for regular
> compiles.
This depends on the Python version. PyObject_DEL (which really should be
PyObject_Del, i.e. the function not the macro) is not defined in 1.5 but
it's defined in 2.x and the proper way to free memory allocated with
PyObject_New. In 1.5 PyMem_Del is OK.
If you want compatibility from 1.5.2 up to 2.2 with pymalloc something
like this should work, I think:
#if PY_VERSION_HEX < 0x01060000
#define PyObject_Del(op) PyMem_Del((op))
#endif
This is basically what is done in Python's _sre.c.
Python's memory management API (which is a bit of a mess) and pymalloc
is currently a very actively discussed topic on python-dev.
Bernhard
--
Intevation GmbH http://intevation.de/
Sketch http://sketch.sourceforge.net/
MapIt! http://www.mapit.de/
