[PATCH] Fix accessing a function's fields (parameters) from Python (PR 18073)

Joel Brobecker brobecker@adacore.com
Thu Oct 22 10:06:00 GMT 2015


Hi Simon,

> > Since 7.4, gdb doesn't allow calling .fields() on a function type, even
> > though the documentation states it should return a list corresponding to
> > the function's parameters.  This patch restores the intended behaviour
> > and adds a test for it.
> >
> > Reg-tested on Arch Linux x86-64.
> >
> > gdb/ChangeLog:
> >
> >         PR python/18073
> >         * python/py-type.c (typy_get_composite): Allow returning a
> >         function type.
> >
> > gdb/testsuite/ChangeLog:
> >
> >         PR python/18073
> >         * gdb.python/py-type.c (C::a_method): New.
> >         (C::a_const_method): New.
> >         (C::a_static_method): New.
> >         (a_function): New.
> >         * gdb.python/py-type.exp (test_fields): Test getting fields
> >         from function and method.

The patch looks good to me, although I am not one of the main Python
maintainers. Normally, I'd ask the author to wait an extra week for
additional comments, but since you've already waited a month, I think
it's only fair to give you the go ahead to push. If there are comments
with your code, we can handle them then.

Thanks also adding the tests. You probably know this, but this is
always appreciated.

Thanks!

-- 
Joel



More information about the Gdb-patches mailing list