Remove some unneeded casts from remote.c
authorTom Tromey <tom@tromey.com>
Thu, 8 Sep 2016 15:49:07 +0000 (09:49 -0600)
committerTom Tromey <tom@tromey.com>
Thu, 8 Sep 2016 15:50:28 +0000 (09:50 -0600)
I happened to notice a few unneeded casts in remote.c.  In some cases
these are no-ops, and in others these cast away const, but in a context
where this is not needed.

I'm checking this in under the obvious rule.
Tested by rebuilding on x86-64 Fedora 24.

2016-09-08  Tom Tromey  <tom@tromey.com>

* remote.c (remote_notif_stop_ack, remote_wait_as)
(show_remote_cmd): Remove unneeded casts.

gdb/ChangeLog
gdb/remote.c

index 108528d7bbdb5f3c94229f326e3a27ee93adf38a..e7cc7122d8c12dd820650e2472836d0835b6d3c2 100644 (file)
@@ -1,3 +1,8 @@
+2016-09-08  Tom Tromey  <tom@tromey.com>
+
+       * remote.c (remote_notif_stop_ack, remote_wait_as)
+       (show_remote_cmd): Remove unneeded casts.
+
 2016-09-06  Pedro Alves  <palves@redhat.com>
 
        * top.c (wait_sync_command_done): Don't assume current_ui doesn't
index 910ac813860801255aa3d88193c717f657da5a05..13258b9eb47e775486936c838547db0899e80e51 100644 (file)
@@ -6029,7 +6029,7 @@ remote_notif_stop_ack (struct notif_client *self, char *buf,
   struct stop_reply *stop_reply = (struct stop_reply *) event;
 
   /* acknowledge */
-  putpkt ((char *) self->ack_command);
+  putpkt (self->ack_command);
 
   if (stop_reply->ws.kind == TARGET_WAITKIND_IGNORE)
       /* We got an unknown stop reply.  */
@@ -6979,8 +6979,8 @@ remote_wait_as (ptid_t ptid, struct target_waitstatus *status, int options)
          rs->last_sent_signal = GDB_SIGNAL_0;
          target_terminal_inferior ();
 
-         strcpy ((char *) buf, rs->last_sent_step ? "s" : "c");
-         putpkt ((char *) buf);
+         strcpy (buf, rs->last_sent_step ? "s" : "c");
+         putpkt (buf);
          break;
        }
       /* else fallthrough */
@@ -13395,7 +13395,7 @@ show_remote_cmd (char *args, int from_tty)
        ui_out_field_string (uiout, "name", list->name);
        ui_out_text (uiout, ":  ");
        if (list->type == show_cmd)
-         do_show_command ((char *) NULL, from_tty, list);
+         do_show_command (NULL, from_tty, list);
        else
          cmd_func (list, NULL, from_tty);
        /* Close the tuple.  */