From 3c9ebddd93ffb8b44b8cc69f3be9db08c861368e Mon Sep 17 00:00:00 2001 From: Tom Tromey Date: Sun, 30 Apr 2017 21:17:58 -0600 Subject: [PATCH] Replace do_restore_instream_cleanup with scoped_restore This changes the users of do_restore_instream_cleanup to use a scoped_restore instead. This patch is broken out because it warrants some additional attention: in particular it's unclear to me whether current_ui can change in the body of these functions -- but if it can, then the cleanup would have modified a different UI's instream member. ChangeLog 2017-08-03 Tom Tromey * top.h (do_restore_instream_cleanup): Remove. * top.c (do_restore_instream_cleanup): Remove. (read_command_file): Use scoped_restore. * cli/cli-script.c (execute_user_command): Use scoped_restore. --- gdb/ChangeLog | 7 +++++++ gdb/cli/cli-script.c | 6 ++---- gdb/top.c | 19 ++----------------- gdb/top.h | 2 -- 4 files changed, 11 insertions(+), 23 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 93373659735..4459acc4bf1 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,10 @@ +2017-08-03 Tom Tromey + + * top.h (do_restore_instream_cleanup): Remove. + * top.c (do_restore_instream_cleanup): Remove. + (read_command_file): Use scoped_restore. + * cli/cli-script.c (execute_user_command): Use scoped_restore. + 2017-08-03 Tom Tromey * cli/cli-script.c (execute_user_command) diff --git a/gdb/cli/cli-script.c b/gdb/cli/cli-script.c index d8dfbfc0971..4b8ae0b1dca 100644 --- a/gdb/cli/cli-script.c +++ b/gdb/cli/cli-script.c @@ -377,7 +377,6 @@ execute_user_command (struct cmd_list_element *c, char *args) { struct ui *ui = current_ui; struct command_line *cmdlines; - struct cleanup *old_chain; enum command_control_type ret; extern unsigned int max_user_call_depth; @@ -393,8 +392,8 @@ execute_user_command (struct cmd_list_element *c, char *args) /* Set the instream to 0, indicating execution of a user-defined function. */ - old_chain = make_cleanup (do_restore_instream_cleanup, ui->instream); - ui->instream = NULL; + scoped_restore restore_instream + = make_scoped_restore (&ui->instream, nullptr); scoped_restore save_async = make_scoped_restore (¤t_ui->async, 0); @@ -410,7 +409,6 @@ execute_user_command (struct cmd_list_element *c, char *args) } cmdlines = cmdlines->next; } - do_cleanups (old_chain); } /* This function is called every time GDB prints a prompt. It ensures diff --git a/gdb/top.c b/gdb/top.c index 43bf0d42d41..6b00c6e7c2c 100644 --- a/gdb/top.c +++ b/gdb/top.c @@ -444,27 +444,14 @@ quit_cover (void) event-top.c into this file, top.c. */ /* static */ const char *source_file_name; -/* Clean up on error during a "source" command (or execution of a - user-defined command). */ - -void -do_restore_instream_cleanup (void *stream) -{ - struct ui *ui = current_ui; - - /* Restore the previous input stream. */ - ui->instream = (FILE *) stream; -} - /* Read commands from STREAM. */ void read_command_file (FILE *stream) { struct ui *ui = current_ui; - struct cleanup *cleanups; - cleanups = make_cleanup (do_restore_instream_cleanup, ui->instream); - ui->instream = stream; + scoped_restore save_instream + = make_scoped_restore (&ui->instream, stream); /* Read commands from `instream' and execute them until end of file or error reading instream. */ @@ -479,8 +466,6 @@ read_command_file (FILE *stream) break; command_handler (command); } - - do_cleanups (cleanups); } void (*pre_init_ui_hook) (void); diff --git a/gdb/top.h b/gdb/top.h index 5d7cb1f8108..45798897f69 100644 --- a/gdb/top.h +++ b/gdb/top.h @@ -290,8 +290,6 @@ extern void show_history (char *, int); extern void set_verbose (char *, int, struct cmd_list_element *); -extern void do_restore_instream_cleanup (void *stream); - extern char *handle_line_of_input (struct buffer *cmd_line_buffer, char *rl, int repeat, const char *annotation_suffix); -- 2.30.2