On Fri, 2011-06-24 at 14:14 -0600, Tom Tromey wrote:
David> Note to self: I see that you posted the resulting gdb patch
here:
David>
http://sourceware.org/ml/gdb-patches/2011-06/msg00376.html
David> (Obviously I'm interested in bugs found and fixed using this tool!)
I thought of CCing this list but in the end just CCd you personally.
But I am happy to CC the plugin list for future changes...
Unfortunately I've just run into a good reason for sending such patches
to this list: I fear that the checker may have given you false results,
and that there might thus be a bug in the gdb patch above.
Specifically, the PyArg_Parse* checker had a bug in how it coped with
the various "#" codes: it assumed that the code had a
#define PY_SSIZE_T_CLEAN
before the initial
#include <Python.h>
The presence or absence of this macro affects all of the "#" format
codes: without it, they should be "int", with it, they should be
"Py_ssize_t". (This is documented much better in the py3k version of
the docs [1], than in the python 2 version [2]).
I believe I've now fixed it in git, with:
http://git.fedorahosted.org/git/?p=gcc-python-plugin.git;a=commit;h=ad8af...
Sorry about this
Dave
[1]
http://docs.python.org/py3k/c-api/arg.html
[2]
http://docs.python.org/c-api/arg.html