diff options
| author | Benno Fünfstück <benno.fuenfstueck@gmail.com> | 2018-11-21 17:06:05 +0000 |
|---|---|---|
| committer | Pedro Alves <palves@redhat.com> | 2018-11-21 17:06:05 +0000 |
| commit | 1c97054b87495b008c6028d697deff61c9fb0b6e (patch) | |
| tree | b16455eea22893a8da4b3485dc247b7c5b301155 /gdb/python | |
| parent | ca94519e70c39fca1d6ea93b8604349111522a8b (diff) | |
| download | gdb-1c97054b87495b008c6028d697deff61c9fb0b6e.zip gdb-1c97054b87495b008c6028d697deff61c9fb0b6e.tar.gz gdb-1c97054b87495b008c6028d697deff61c9fb0b6e.tar.bz2 | |
Make command-repeat work after gdb.execute
Since commit
56bcdbea2bed ("Let gdb.execute handle multi-line commands")
command repetition after using the `gdb.execute` Python function
fails (the previous command is not repeated anymore). This happens
because read_command_lines_1 sets dont_repeat, but the call to
prevent_dont_repeat in execute_gdb_command is later.
The fix is to move the call to prevent_dont_repeat to the beginning of
the function.
Tested on my laptop (ArchLinux-x86_64).
gdb/ChangeLog:
PR python/23714
* gdb/python/python.c (execute_gdb_command): Call
prevent_dont_repeat earlier to avoid affecting dont_repeat.
gdb/testuite/ChangeLog:
PR python/23714
* gdb.python/python.exp: Test command repetition after
gdb.execute.
Diffstat (limited to 'gdb/python')
| -rw-r--r-- | gdb/python/python.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/gdb/python/python.c b/gdb/python/python.c index a37ed10..d6453e7 100644 --- a/gdb/python/python.c +++ b/gdb/python/python.c @@ -583,6 +583,8 @@ execute_gdb_command (PyObject *self, PyObject *args, PyObject *kw) std::string to_string_res; + scoped_restore preventer = prevent_dont_repeat (); + TRY { struct interp *interp; @@ -612,7 +614,6 @@ execute_gdb_command (PyObject *self, PyObject *args, PyObject *kw) interp = interp_lookup (current_ui, "console"); current_uiout = interp->interp_ui_out (); - scoped_restore preventer = prevent_dont_repeat (); if (to_string) to_string_res = execute_control_commands_to_string (lines.get (), from_tty); |