[1/2] gdb: remove interp_pre_command_loop
Checks
Context |
Check |
Description |
linaro-tcwg-bot/tcwg_gdb_build--master-aarch64 |
success
|
Testing passed
|
linaro-tcwg-bot/tcwg_gdb_build--master-arm |
success
|
Testing passed
|
linaro-tcwg-bot/tcwg_gdb_check--master-arm |
success
|
Testing passed
|
linaro-tcwg-bot/tcwg_gdb_check--master-aarch64 |
success
|
Testing passed
|
Commit Message
It is a trivial wrapper around the pre_command_loop method, remove it.
Change-Id: Idb2c61f9b68988528006a9a9b2b528f43781eef4
---
gdb/interps.c | 10 ----------
gdb/interps.h | 4 ----
gdb/main.c | 2 +-
gdb/ui.c | 2 +-
4 files changed, 2 insertions(+), 16 deletions(-)
base-commit: 313b2841b8e9046ea658104988e01bedf6148d5f
Comments
>>>>> "Simon" == Simon Marchi via Gdb-patches <gdb-patches@sourceware.org> writes:
Simon> It is a trivial wrapper around the pre_command_loop method, remove it.
Makes sense to me.
Approved-By: Tom Tromey <tom@tromey.com>
Tom
On 9/7/23 14:03, Tom Tromey wrote:
>>>>>> "Simon" == Simon Marchi via Gdb-patches <gdb-patches@sourceware.org> writes:
>
> Simon> It is a trivial wrapper around the pre_command_loop method, remove it.
>
> Makes sense to me.
> Approved-By: Tom Tromey <tom@tromey.com>
>
> Tom
Thanks, pushed both patches.
Simon
@@ -262,16 +262,6 @@ command_interp (void)
return current_ui->current_interpreter;
}
-/* See interps.h. */
-
-void
-interp_pre_command_loop (struct interp *interp)
-{
- gdb_assert (interp != NULL);
-
- interp->pre_command_loop ();
-}
-
/* See interp.h */
int
@@ -261,10 +261,6 @@ extern void clear_interpreter_hooks (void);
if it uses GDB's own simplified form of readline. */
extern int interp_supports_command_editing (struct interp *interp);
-/* Called before starting an event loop, to give the interpreter a
- chance to e.g., print a prompt. */
-extern void interp_pre_command_loop (struct interp *interp);
-
/* List the possible interpreters which could complete the given
text. */
extern void interpreter_completer (struct cmd_list_element *ignore,
@@ -470,7 +470,7 @@ captured_command_loop ()
/* Give the interpreter a chance to print a prompt, if necessary */
if (ui->prompt_state != PROMPT_BLOCKED)
- interp_pre_command_loop (top_level_interpreter ());
+ top_level_interpreter ()->pre_command_loop ();
/* Now it's time to start the event loop. */
start_event_loop ();
@@ -227,7 +227,7 @@ new_ui_command (const char *args, int from_tty)
set_top_level_interpreter (interpreter_name);
- interp_pre_command_loop (top_level_interpreter ());
+ top_level_interpreter ()->pre_command_loop ();
/* Make sure the file is not closed. */
stream.release ();