Remove an unneeded NULL check
[deliverable/binutils-gdb.git] / gdb / tui / tui-layout.c
index 695c56012c1c8aed61ab805f79c4d097f90ed306..de78c6c0ac8fa48727ccd7e853f681fb7f558716 100644 (file)
@@ -49,12 +49,11 @@ static void init_gen_win_info (struct tui_gen_win_info *,
 static void *init_and_make_win (void *, enum tui_win_type, 
                                int, int, int, int, int);
 static void show_source_or_disasm_and_command (enum tui_layout_type);
-static void make_source_or_disasm_window (struct tui_win_info **, 
-                                         enum tui_win_type, 
-                                         int, int);
-static void make_command_window (struct tui_win_info **, int, int);
-static void make_source_window (struct tui_win_info **, int, int);
-static void make_disasm_window (struct tui_win_info **, int, int);
+static struct tui_win_info *make_source_or_disasm_window (enum tui_win_type, 
+                                                         int, int);
+static struct tui_win_info *make_command_window (int, int);
+static struct tui_win_info *make_source_window (int, int);
+static struct tui_win_info *make_disasm_window (int, int);
 static void make_data_window (struct tui_win_info **, int, int);
 static void show_source_command (void);
 static void show_disasm_command (void);
@@ -228,11 +227,7 @@ tui_set_layout (enum tui_layout_type layout_type)
          tui_update_source_windows_with_addr (gdbarch, addr);
 
          if (regs_populate)
-           {
-             struct reggroup *group =
-               TUI_DATA_WIN->detail.data_display_info.current_group;
-             tui_show_registers (group);
-           }
+           tui_show_registers (TUI_DATA_WIN->current_group);
        }
     }
   else
@@ -469,17 +464,17 @@ extract_display_start_addr (struct gdbarch **gdbarch_p, CORE_ADDR *addr_p)
     {
     case SRC_COMMAND:
     case SRC_DATA_COMMAND:
-      gdbarch = TUI_SRC_WIN->detail.source_info.gdbarch;
+      gdbarch = TUI_SRC_WIN->gdbarch;
       find_line_pc (cursal.symtab,
-                   TUI_SRC_WIN->detail.source_info.start_line_or_addr.u.line_no,
+                   TUI_SRC_WIN->start_line_or_addr.u.line_no,
                    &pc);
       addr = pc;
       break;
     case DISASSEM_COMMAND:
     case SRC_DISASSEM_COMMAND:
     case DISASSEM_DATA_COMMAND:
-      gdbarch = TUI_DISASM_WIN->detail.source_info.gdbarch;
-      addr = TUI_DISASM_WIN->detail.source_info.start_line_or_addr.u.addr;
+      gdbarch = TUI_DISASM_WIN->gdbarch;
+      addr = TUI_DISASM_WIN->start_line_or_addr.u.addr;
       break;
     default:
       addr = 0;
@@ -540,44 +535,37 @@ prev_layout (void)
 
 
 
-static void
-make_command_window (struct tui_win_info **win_info_ptr, 
-                    int height, int origin_y)
+static struct tui_win_info *
+make_command_window (int height, int origin_y)
 {
-  *win_info_ptr
-    = (struct tui_win_info *) init_and_make_win (*win_info_ptr,
+  struct tui_win_info *result
+    = (struct tui_win_info *) init_and_make_win (NULL,
                                                 CMD_WIN,
                                                 height,
                                                 tui_term_width (),
                                                 0,
                                                 origin_y,
                                                 DONT_BOX_WINDOW);
-
-  (*win_info_ptr)->can_highlight = FALSE;
+  result->can_highlight = FALSE;
+  return result;
 }
 
 
 /* make_source_window().
  */
-static void
-make_source_window (struct tui_win_info **win_info_ptr, 
-                   int height, int origin_y)
+static struct tui_win_info *
+make_source_window (int height, int origin_y)
 {
-  make_source_or_disasm_window (win_info_ptr, SRC_WIN, height, origin_y);
-
-  return;
+  return make_source_or_disasm_window (SRC_WIN, height, origin_y);
 }                              /* make_source_window */
 
 
 /* make_disasm_window().
  */
-static void
-make_disasm_window (struct tui_win_info **win_info_ptr, 
-                   int height, int origin_y)
+static struct tui_win_info *
+make_disasm_window (int height, int origin_y)
 {
-  make_source_or_disasm_window (win_info_ptr, DISASSEM_WIN, height, origin_y);
-
-  return;
+  return make_source_or_disasm_window (DISASSEM_WIN, height, origin_y);
 }                              /* make_disasm_window */
 
 
@@ -630,96 +618,90 @@ show_source_disasm_command (void)
       asm_height = tui_term_height () - (src_height + cmd_height);
 
       if (TUI_SRC_WIN == NULL)
-       make_source_window (&TUI_SRC_WIN, src_height, 0);
+       tui_win_list[SRC_WIN] = make_source_window (src_height, 0);
       else
        {
          init_gen_win_info (&TUI_SRC_WIN->generic,
                             TUI_SRC_WIN->generic.type,
                             src_height,
                             TUI_SRC_WIN->generic.width,
-                            TUI_SRC_WIN->detail.source_info.execution_info->width,
+                            TUI_SRC_WIN->execution_info->width,
                             0);
          TUI_SRC_WIN->can_highlight = TRUE;
-         init_gen_win_info (TUI_SRC_WIN->detail.source_info.execution_info,
+         init_gen_win_info (TUI_SRC_WIN->execution_info,
                             EXEC_INFO_WIN,
                             src_height,
                             3,
                             0,
                             0);
          tui_make_visible (&TUI_SRC_WIN->generic);
-         tui_make_visible (TUI_SRC_WIN->detail.source_info.execution_info);
-         TUI_SRC_WIN->detail.source_info.has_locator = FALSE;;
+         tui_make_visible (TUI_SRC_WIN->execution_info);
+         TUI_SRC_WIN->m_has_locator = false;
        }
-      if (TUI_SRC_WIN != NULL)
-       {
-         struct tui_gen_win_info *locator = tui_locator_win_info_ptr ();
 
-         tui_show_source_content (TUI_SRC_WIN);
-         if (TUI_DISASM_WIN == NULL)
-           {
-             make_disasm_window (&TUI_DISASM_WIN, asm_height, src_height - 1);
-             locator
-               = ((struct tui_gen_win_info *)
-                  init_and_make_win (locator,
-                                     LOCATOR_WIN,
-                                     2 /* 1 */ ,
-                                     tui_term_width (),
-                                     0,
-                                     (src_height + asm_height) - 1,
-                                     DONT_BOX_WINDOW));
-           }
-         else
-           {
-             init_gen_win_info (locator,
-                                LOCATOR_WIN,
-                                2 /* 1 */ ,
-                                tui_term_width (),
-                                0,
-                                (src_height + asm_height) - 1);
-             TUI_DISASM_WIN->detail.source_info.has_locator = TRUE;
-             init_gen_win_info (&TUI_DISASM_WIN->generic,
-                                TUI_DISASM_WIN->generic.type,
-                                asm_height,
-                                TUI_DISASM_WIN->generic.width,
-                                TUI_DISASM_WIN->detail.source_info.execution_info->width,
-                                src_height - 1);
-             init_gen_win_info (TUI_DISASM_WIN->detail.source_info.execution_info,
-                                EXEC_INFO_WIN,
-                                asm_height,
-                                3,
-                                0,
-                                src_height - 1);
-             TUI_DISASM_WIN->can_highlight = TRUE;
-             tui_make_visible (&TUI_DISASM_WIN->generic);
-             tui_make_visible (TUI_DISASM_WIN->detail.source_info.execution_info);
-           }
-         if (TUI_DISASM_WIN != NULL)
-           {
-             TUI_SRC_WIN->detail.source_info.has_locator = FALSE;
-             TUI_DISASM_WIN->detail.source_info.has_locator = TRUE;
-             tui_make_visible (locator);
-             tui_show_locator_content ();
-             tui_show_source_content (TUI_DISASM_WIN);
-
-             if (TUI_CMD_WIN == NULL)
-               make_command_window (&TUI_CMD_WIN,
-                                    cmd_height,
-                                    tui_term_height () - cmd_height);
-             else
-               {
-                 init_gen_win_info (&TUI_CMD_WIN->generic,
-                                    TUI_CMD_WIN->generic.type,
-                                    TUI_CMD_WIN->generic.height,
-                                    TUI_CMD_WIN->generic.width,
-                                    0,
-                                    TUI_CMD_WIN->generic.origin.y);
-                 TUI_CMD_WIN->can_highlight = FALSE;
-                 tui_make_visible (&TUI_CMD_WIN->generic);
-               }
-             if (TUI_CMD_WIN != NULL)
-               tui_refresh_win (&TUI_CMD_WIN->generic);
-           }
+      struct tui_gen_win_info *locator = tui_locator_win_info_ptr ();
+
+      tui_show_source_content (TUI_SRC_WIN);
+      if (TUI_DISASM_WIN == NULL)
+       {
+         tui_win_list[DISASSEM_WIN]
+           = make_disasm_window (asm_height, src_height - 1);
+         locator
+           = ((struct tui_gen_win_info *)
+              init_and_make_win (locator,
+                                 LOCATOR_WIN,
+                                 2 /* 1 */ ,
+                                 tui_term_width (),
+                                 0,
+                                 (src_height + asm_height) - 1,
+                                 DONT_BOX_WINDOW));
        }
+      else
+       {
+         init_gen_win_info (locator,
+                            LOCATOR_WIN,
+                            2 /* 1 */ ,
+                            tui_term_width (),
+                            0,
+                            (src_height + asm_height) - 1);
+         TUI_DISASM_WIN->m_has_locator = true;
+         init_gen_win_info (&TUI_DISASM_WIN->generic,
+                            TUI_DISASM_WIN->generic.type,
+                            asm_height,
+                            TUI_DISASM_WIN->generic.width,
+                            TUI_DISASM_WIN->execution_info->width,
+                            src_height - 1);
+         init_gen_win_info (TUI_DISASM_WIN->execution_info,
+                            EXEC_INFO_WIN,
+                            asm_height,
+                            3,
+                            0,
+                            src_height - 1);
+         TUI_DISASM_WIN->can_highlight = TRUE;
+         tui_make_visible (&TUI_DISASM_WIN->generic);
+         tui_make_visible (TUI_DISASM_WIN->execution_info);
+       }
+      TUI_SRC_WIN->m_has_locator = false;
+      TUI_DISASM_WIN->m_has_locator = true;
+      tui_make_visible (locator);
+      tui_show_locator_content ();
+      tui_show_source_content (TUI_DISASM_WIN);
+
+      if (TUI_CMD_WIN == NULL)
+       tui_win_list[CMD_WIN]
+         = make_command_window (cmd_height, tui_term_height () - cmd_height);
+      else
+       {
+         init_gen_win_info (&TUI_CMD_WIN->generic,
+                            TUI_CMD_WIN->generic.type,
+                            TUI_CMD_WIN->generic.height,
+                            TUI_CMD_WIN->generic.width,
+                            0,
+                            TUI_CMD_WIN->generic.origin.y);
+         TUI_CMD_WIN->can_highlight = FALSE;
+         tui_make_visible (&TUI_CMD_WIN->generic);
+       }
+      tui_refresh_win (&TUI_CMD_WIN->generic);
       tui_set_current_layout_to (SRC_DISASSEM_COMMAND);
     }
 }
@@ -740,18 +722,22 @@ show_data (enum tui_layout_type new_layout)
   src_height = total_height - data_height;
   tui_make_all_invisible ();
   tui_make_invisible (locator);
-  make_data_window (&TUI_DATA_WIN, data_height, 0);
+  make_data_window (&tui_win_list[DATA_WIN], data_height, 0);
   TUI_DATA_WIN->can_highlight = TRUE;
   if (new_layout == SRC_DATA_COMMAND)
     win_type = SRC_WIN;
   else
     win_type = DISASSEM_WIN;
+
+  tui_source_window_base *base;
   if (tui_win_list[win_type] == NULL)
     {
       if (win_type == SRC_WIN)
-       make_source_window (&tui_win_list[win_type], src_height, data_height - 1);
+       tui_win_list[win_type]
+         = make_source_window (src_height, data_height - 1);
       else
-       make_disasm_window (&tui_win_list[win_type], src_height, data_height - 1);
+       tui_win_list[win_type]
+         = make_disasm_window (src_height, data_height - 1);
       locator
        = ((struct tui_gen_win_info *)
           init_and_make_win (locator,
@@ -761,23 +747,25 @@ show_data (enum tui_layout_type new_layout)
                              0,
                              total_height - 1,
                              DONT_BOX_WINDOW));
+      base = (tui_source_window_base *) tui_win_list[win_type];
     }
   else
     {
+      base = (tui_source_window_base *) tui_win_list[win_type];
       init_gen_win_info (&tui_win_list[win_type]->generic,
                         tui_win_list[win_type]->generic.type,
                         src_height,
                         tui_win_list[win_type]->generic.width,
-                        tui_win_list[win_type]->detail.source_info.execution_info->width,
+                        base->execution_info->width,
                         data_height - 1);
-      init_gen_win_info (tui_win_list[win_type]->detail.source_info.execution_info,
+      init_gen_win_info (base->execution_info,
                         EXEC_INFO_WIN,
                         src_height,
                         3,
                         0,
                         data_height - 1);
       tui_make_visible (&tui_win_list[win_type]->generic);
-      tui_make_visible (tui_win_list[win_type]->detail.source_info.execution_info);
+      tui_make_visible (base->execution_info);
       init_gen_win_info (locator,
                         LOCATOR_WIN,
                         2 /* 1 */ ,
@@ -785,7 +773,7 @@ show_data (enum tui_layout_type new_layout)
                         0,
                         total_height - 1);
     }
-  tui_win_list[win_type]->detail.source_info.has_locator = TRUE;
+  base->m_has_locator = true;
   tui_make_visible (locator);
   tui_show_locator_content ();
   tui_add_to_source_windows (tui_win_list[win_type]);
@@ -856,9 +844,8 @@ init_and_make_win (void *opaque_win_info,
 }
 
 
-static void
-make_source_or_disasm_window (struct tui_win_info **win_info_ptr, 
-                             enum tui_win_type type,
+static struct tui_win_info *
+make_source_or_disasm_window (enum tui_win_type type,
                              int height, int origin_y)
 {
   struct tui_gen_win_info *execution_info = NULL;
@@ -879,17 +866,17 @@ make_source_or_disasm_window (struct tui_win_info **win_info_ptr,
                          DONT_BOX_WINDOW));
 
   /* Now create the source window.  */
-  *win_info_ptr
-    = ((struct tui_win_info *)
-       init_and_make_win (*win_info_ptr,
+  struct tui_source_window_base *result
+    = ((struct tui_source_window_base *)
+       init_and_make_win (NULL,
                          type,
                          height,
                          tui_term_width () - execution_info->width,
                          execution_info->width,
                          origin_y,
                          BOX_WINDOW));
-
-  (*win_info_ptr)->detail.source_info.execution_info = execution_info;
+  result->execution_info = execution_info;
+  return result;
 }
 
 
@@ -910,16 +897,17 @@ show_source_or_disasm_and_command (enum tui_layout_type layout_type)
       src_height = tui_term_height () - cmd_height;
 
       if (layout_type == SRC_COMMAND)
-       win_info_ptr = &TUI_SRC_WIN;
+       win_info_ptr = &tui_win_list[SRC_WIN];
       else
-       win_info_ptr = &TUI_DISASM_WIN;
+       win_info_ptr = &tui_win_list[DISASSEM_WIN];
 
+      tui_source_window_base *base;
       if ((*win_info_ptr) == NULL)
        {
          if (layout_type == SRC_COMMAND)
-           make_source_window (win_info_ptr, src_height - 1, 0);
+           *win_info_ptr = make_source_window (src_height - 1, 0);
          else
-           make_disasm_window (win_info_ptr, src_height - 1, 0);
+           *win_info_ptr = make_disasm_window (src_height - 1, 0);
          locator
            = ((struct tui_gen_win_info *)
               init_and_make_win (locator,
@@ -929,55 +917,56 @@ show_source_or_disasm_and_command (enum tui_layout_type layout_type)
                                  0,
                                  src_height - 1,
                                  DONT_BOX_WINDOW));
+         base = (tui_source_window_base *) *win_info_ptr;
        }
       else
        {
+         base = (tui_source_window_base *) *win_info_ptr;
          init_gen_win_info (locator,
                             LOCATOR_WIN,
                             2 /* 1 */ ,
                             tui_term_width (),
                             0,
                             src_height - 1);
-         (*win_info_ptr)->detail.source_info.has_locator = TRUE;
+         base->m_has_locator = true;
          init_gen_win_info (&(*win_info_ptr)->generic,
                             (*win_info_ptr)->generic.type,
                             src_height - 1,
                             (*win_info_ptr)->generic.width,
-                            (*win_info_ptr)->detail.source_info.execution_info->width,
+                            base->execution_info->width,
                             0);
-         init_gen_win_info ((*win_info_ptr)->detail.source_info.execution_info,
+         init_gen_win_info (base->execution_info,
                             EXEC_INFO_WIN,
                             src_height - 1,
                             3,
                             0,
                             0);
-         (*win_info_ptr)->can_highlight = TRUE;
+         base->can_highlight = TRUE;
          tui_make_visible (&(*win_info_ptr)->generic);
-         tui_make_visible ((*win_info_ptr)->detail.source_info.execution_info);
+         tui_make_visible (base->execution_info);
        }
-      if ((*win_info_ptr) != NULL)
-       {
-         (*win_info_ptr)->detail.source_info.has_locator = TRUE;
-         tui_make_visible (locator);
-         tui_show_locator_content ();
-         tui_show_source_content (*win_info_ptr);
 
-         if (TUI_CMD_WIN == NULL)
-           {
-             make_command_window (&TUI_CMD_WIN, cmd_height, src_height);
-             tui_refresh_win (&TUI_CMD_WIN->generic);
-           }
-         else
-           {
-             init_gen_win_info (&TUI_CMD_WIN->generic,
-                                TUI_CMD_WIN->generic.type,
-                                TUI_CMD_WIN->generic.height,
-                                TUI_CMD_WIN->generic.width,
-                                TUI_CMD_WIN->generic.origin.x,
-                                TUI_CMD_WIN->generic.origin.y);
-             TUI_CMD_WIN->can_highlight = FALSE;
-             tui_make_visible (&TUI_CMD_WIN->generic);
-           }
+      base->m_has_locator = true;
+      tui_make_visible (locator);
+      tui_show_locator_content ();
+      tui_show_source_content (*win_info_ptr);
+
+      if (TUI_CMD_WIN == NULL)
+       {
+         tui_win_list[CMD_WIN] = make_command_window (cmd_height,
+                                                      src_height);
+         tui_refresh_win (&TUI_CMD_WIN->generic);
+       }
+      else
+       {
+         init_gen_win_info (&TUI_CMD_WIN->generic,
+                            TUI_CMD_WIN->generic.type,
+                            TUI_CMD_WIN->generic.height,
+                            TUI_CMD_WIN->generic.width,
+                            TUI_CMD_WIN->generic.origin.x,
+                            TUI_CMD_WIN->generic.origin.y);
+         TUI_CMD_WIN->can_highlight = FALSE;
+         tui_make_visible (&TUI_CMD_WIN->generic);
        }
       tui_set_current_layout_to (layout_type);
     }
This page took 0.029075 seconds and 4 git commands to generate.