Toolchain/binutils-gdb
1
0
Fork 0
forked from Imagelibrary/binutils-gdb
Code Pull Requests Activity

py-breakpoint: Don't use the 'p' PyArg_ParseTupleAndKeywords format specifier

Browse Source 
In Python 3, the 'p' format specifier can be passed to
PyArg_ParseTupleAndKeywords to test the argument for truth and convert
it to a boolean value (the p stands for predicate).  However, it is not
available in Python 2, causing this error:

  Traceback (most recent call last):
    File "test.py", line 1, in <module>
      b1 = gdb.Breakpoint("foo", qualified=False)
  TypeError: argument 10 (impossible<bad format char>)

This patch changes it to the 'O' specifier, which returns the Python
object passed in without transformation, and uses PyObject_IsTrue on it.
This is what is done for the other boolean parameters of this function
(internal and temporary).

This fixes the test gdb.python/py-breakpoint.exp for Python 2.

gdb/ChangeLog:

	* python/py-breakpoint.c (bppy_init): Use 'O' format specifier
	for "qualified" and use PyObject_IsTrue.
This commit is contained in:
Simon Marchi
2017-12-14 15:46:47 -05:00
parent 863f7a5f48
commit 8e557e52b3
2 changed files with 8 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
gdb/ChangeLog
View File

@@ -1,3 +1,8 @@
2017-12-14 Simon Marchi <simon.marchi@ericsson.com>
* python/py-breakpoint.c (bppy_init): Use 'O' format specifier
for "qualified" and use PyObject_IsTrue.
2017-12-14 Jan Kratochvil <jan.kratochvil@redhat.com> 2017-12-14 Jan Kratochvil <jan.kratochvil@redhat.com>
* dwarf2read.c (dw2_debug_names_iterator::next): Support * dwarf2read.c (dw2_debug_names_iterator::next): Support

6
gdb/python/py-breakpoint.c
View File

@@ -707,9 +707,9 @@ bppy_init (PyObject *self, PyObject *args, PyObject *kwargs)
char *label = NULL; char *label = NULL;
char *source = NULL; char *source = NULL;
char *function = NULL; char *function = NULL;
int qualified = 0; PyObject * qualified = NULL;
if (!gdb_PyArg_ParseTupleAndKeywords (args, kwargs, "|siiOOsssOp", keywords, if (!gdb_PyArg_ParseTupleAndKeywords (args, kwargs, "|siiOOsssOO", keywords,
&spec, &type, &access_type, &spec, &type, &access_type,
&internal, &internal,
&temporary, &source, &temporary, &source,
@@ -762,7 +762,7 @@ bppy_init (PyObject *self, PyObject *args, PyObject *kwargs)
{ {
event_location_up location; event_location_up location;
symbol_name_match_type func_name_match_type symbol_name_match_type func_name_match_type
= (qualified = (qualified != NULL && PyObject_IsTrue (qualified)
? symbol_name_match_type::FULL ? symbol_name_match_type::FULL
: symbol_name_match_type::WILD); : symbol_name_match_type::WILD);