gdb/python: don't use the 'p' format for parsing args
authorAndrew Burgess <aburgess@redhat.com>
Mon, 29 Nov 2021 13:53:06 +0000 (13:53 +0000)
committerAndrew Burgess <aburgess@redhat.com>
Tue, 30 Nov 2021 15:46:09 +0000 (15:46 +0000)
commit90fe61ced1c9aa4afb263326e336330d15603fbf
treecd3361cec8660cd59e00766142f010d119c2eef8
parentd0b39fc48552fb96b00176145f11ecc871ef4024
gdb/python: don't use the 'p' format for parsing args

When running the gdb.python/py-arch.exp tests on a GDB built
against Python 2 I ran into some errors.  The problem is that this
test script exercises the gdb.Architecture.integer_type method, and
this method uses 'p' as an argument format specifier in a call to
gdb_PyArg_ParseTupleAndKeywords.

Unfortunately this specified was only added in Python 3.3, so will
cause an error for earlier versions of Python.

This commit switches to use the 'O' specifier to collect a PyObject,
and then uses PyObject_IsTrue to convert the object to a boolean.

An earlier version of this patch incorrectly switched from using 'p'
to use 'i', however, it was pointed out during review that this would
cause some changes in behaviour, for example both of these will work
with 'p', but not with 'i':

  gdb.selected_inferior().architecture().integer_type(32, None)
  gdb.selected_inferior().architecture().integer_type(32, "foo")

The new approach of using 'O' works fine with these cases.  I've added
some new tests to cover both of the above.

There should be no user visible changes after this commit.
gdb/python/py-arch.c
gdb/testsuite/gdb.python/py-arch.exp