Change tui_get_register to return void
authorTom Tromey <tom@tromey.com>
Mon, 1 Jul 2019 06:51:20 +0000 (00:51 -0600)
committerTom Tromey <tom@tromey.com>
Wed, 17 Jul 2019 18:19:17 +0000 (12:19 -0600)
tui_get_register returns a tui_status, but nothing checks this, so it
can return void instead.

gdb/ChangeLog
2019-07-17  Tom Tromey  <tom@tromey.com>

* tui/tui-regs.c (tui_get_register): Return void.

gdb/ChangeLog
gdb/tui/tui-regs.c

index 0de9c33..8de1558 100644 (file)
@@ -1,5 +1,9 @@
 2019-07-17  Tom Tromey  <tom@tromey.com>
 
+       * tui/tui-regs.c (tui_get_register): Return void.
+
+2019-07-17  Tom Tromey  <tom@tromey.com>
+
        * tui/tui-wingeneral.c (tui_gen_win_info::make_visible):
        Simplify.
 
index 382a441..52f6702 100644 (file)
@@ -54,9 +54,9 @@ static void tui_show_register_group (tui_data_window *win_info,
                                     struct frame_info *frame,
                                     int refresh_values_only);
 
-static enum tui_status tui_get_register (struct frame_info *frame,
-                                        struct tui_data_item_window *data,
-                                        int regnum, bool *changedp);
+static void tui_get_register (struct frame_info *frame,
+                             struct tui_data_item_window *data,
+                             int regnum, bool *changedp);
 
 
 
@@ -646,13 +646,11 @@ tui_register_format (struct frame_info *frame, int regnum)
 /* Get the register value from the given frame and format it for the
    display.  When changep is set, check if the new register value has
    changed with respect to the previous call.  */
-static enum tui_status
+static void
 tui_get_register (struct frame_info *frame,
                   struct tui_data_item_window *data, 
                  int regnum, bool *changedp)
 {
-  enum tui_status ret = TUI_FAILURE;
-
   if (changedp)
     *changedp = false;
   if (target_has_registers)
@@ -666,8 +664,5 @@ tui_get_register (struct frame_info *frame,
        *changedp = true;
 
       xfree (prev_content);
-
-      ret = TUI_SUCCESS;
     }
-  return ret;
 }