[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