From: Pedro Alves Date: Tue, 22 May 2018 17:22:09 +0000 (+0100) Subject: remote: Small cleanup in compare_section_command X-Git-Tag: binutils-2_31~384 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=0e9a6b2f040e016c9dd7f9841a0272e488ed2fd5;p=external%2Fbinutils.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 c17528f..205dd4d 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,5 +1,10 @@ 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): Move higher up in the file. (remote_state) : Store remote_arch_state values diff --git a/gdb/remote.c b/gdb/remote.c index 1d1819b..fbf23b8 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;