From: Pedro Alves Date: Tue, 22 May 2018 17:22:09 +0000 (+0100) Subject: remote: Small cleanup in compare_section_command X-Git-Url: http://drtracing.org/?a=commitdiff_plain;h=0e9a6b2f040e016c9dd7f9841a0272e488ed2fd5;p=deliverable%2Fbinutils-gdb.git remote: Small cleanup in compare_section_command The set_general_process call in compare_sections_command isn't actually needed. remote_target::verify_memory and remote_target::xfer_partial already handle making sure the remote is pointing at the right process or thread. Getting this out of the way helps a bit with further elimination of the remote_state global, because we have to handle the case of a user invoking the command even if not connect to a remote target. gdb/ChangeLog: 2018-05-22 Pedro Alves * remote.c (compare_section_command): Remove set_general_process call. --- diff --git a/gdb/ChangeLog b/gdb/ChangeLog index c17528ff4c..205dd4d388 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,8 @@ +2018-05-22 Pedro Alves + + * remote.c (compare_section_command): Remove set_general_process + call. + 2018-05-22 Pedro Alves * remote.c (struct packet_reg, struct remote_arch_state): diff --git a/gdb/remote.c b/gdb/remote.c index 1d1819bac1..fbf23b8a43 100644 --- a/gdb/remote.c +++ b/gdb/remote.c @@ -10516,9 +10516,6 @@ compare_sections_command (const char *args, int from_tty) if (!exec_bfd) error (_("command cannot be used without an exec file")); - /* Make sure the remote is pointing at the right process. */ - set_general_process (); - if (args != NULL && strcmp (args, "-r") == 0) { read_only = 1;