Remove unused parameter from two TUI functions
[deliverable/binutils-gdb.git] / gdb / tui / tui-winsource.c
index 204fee13be960cb81284225cce6be21408834728..65aa5164122e97c4c94d47c025e11d86c56237d6 100644 (file)
@@ -94,18 +94,19 @@ tui_update_source_window_as_is (struct tui_source_window_base *win_info,
   enum tui_status ret;
 
   if (win_info->type == SRC_WIN)
-    ret = tui_set_source_content (s, line_or_addr.u.line_no, noerror);
+    ret = tui_set_source_content (win_info, s, line_or_addr.u.line_no,
+                                 noerror);
   else
-    ret = tui_set_disassem_content (gdbarch, line_or_addr.u.addr);
+    ret = tui_set_disassem_content (win_info, gdbarch, line_or_addr.u.addr);
 
   if (ret == TUI_FAILURE)
     {
-      tui_clear_source_content (win_info, EMPTY_SOURCE_PROMPT);
+      tui_clear_source_content (win_info);
       tui_clear_exec_info_content (win_info);
     }
   else
     {
-      tui_update_breakpoint_info (win_info, 0);
+      tui_update_breakpoint_info (win_info, nullptr, false);
       tui_show_source_content (win_info);
       tui_update_exec_info (win_info);
       if (win_info->type == SRC_WIN)
@@ -113,7 +114,7 @@ tui_update_source_window_as_is (struct tui_source_window_base *win_info,
          symtab_and_line sal;
 
          sal.line = line_or_addr.u.line_no +
-           (win_info->content_size - 2);
+           (win_info->content.size () - 2);
          sal.symtab = s;
          sal.pspace = SYMTAB_PSPACE (s);
          set_current_source_symtab_and_line (sal);
@@ -121,7 +122,7 @@ tui_update_source_window_as_is (struct tui_source_window_base *win_info,
             we don't have a split layout.  */
          if (tui_win_with_focus () == TUI_DISASM_WIN
              && tui_current_layout () != SRC_DISASSEM_COMMAND)
-           tui_set_win_focus_to (TUI_SRC_WIN);
+           tui_set_win_focus_to (win_info);
        }
     }
 
@@ -153,7 +154,7 @@ tui_update_source_windows_with_addr (struct gdbarch *gdbarch, CORE_ADDR addr)
          sal = find_pc_line (addr, 0);
          l.loa = LOA_LINE;
          l.u.line_no = sal.line;
-         tui_show_symtab_source (gdbarch, sal.symtab, l, FALSE);
+         tui_show_symtab_source (TUI_SRC_WIN, gdbarch, sal.symtab, l, FALSE);
          break;
        }
     }
@@ -161,7 +162,7 @@ tui_update_source_windows_with_addr (struct gdbarch *gdbarch, CORE_ADDR addr)
     {
       for (struct tui_source_window_base *win_info : tui_source_windows ())
        {
-         tui_clear_source_content (win_info, EMPTY_SOURCE_PROMPT);
+         tui_clear_source_content (win_info);
          tui_clear_exec_info_content (win_info);
        }
     }
@@ -191,7 +192,7 @@ tui_update_source_windows_with_line (struct symtab *s, int line)
     default:
       l.loa = LOA_LINE;
       l.u.line_no = line;
-      tui_show_symtab_source (gdbarch, s, l, FALSE);
+      tui_show_symtab_source (TUI_SRC_WIN, gdbarch, s, l, FALSE);
       if (tui_current_layout () == SRC_DISASSEM_COMMAND)
        {
          find_line_pc (s, line, &pc);
@@ -204,29 +205,27 @@ tui_update_source_windows_with_line (struct symtab *s, int line)
 }
 
 void
-tui_clear_source_content (struct tui_win_info *win_info, 
-                         int display_prompt)
+tui_clear_source_content (struct tui_source_window_base *win_info)
 {
   if (win_info != NULL)
     {
       int i;
 
-      win_info->content_in_use = FALSE;
-      tui_erase_source_content (win_info, display_prompt);
-      for (i = 0; i < win_info->content_size; i++)
+      win_info->content_in_use = false;
+      tui_erase_source_content (win_info);
+      for (i = 0; i < win_info->content.size (); i++)
        {
-         struct tui_win_element *element = win_info->content[i];
+         struct tui_source_element *element = &win_info->content[i];
 
-         element->which_element.source.has_break = FALSE;
-         element->which_element.source.is_exec_point = FALSE;
+         element->break_mode = 0;
+         element->is_exec_point = false;
        }
     }
 }
 
 
 void
-tui_erase_source_content (struct tui_win_info *win_info, 
-                         int display_prompt)
+tui_erase_source_content (struct tui_source_window_base *win_info)
 {
   int x_pos;
   int half_width = (win_info->width - 2) / 2;
@@ -235,30 +234,29 @@ tui_erase_source_content (struct tui_win_info *win_info,
     {
       werase (win_info->handle);
       tui_check_and_display_highlight_if_needed (win_info);
-      if (display_prompt == EMPTY_SOURCE_PROMPT)
-       {
-         const char *no_src_str;
-
-         if (win_info->type == SRC_WIN)
-           no_src_str = NO_SRC_STRING;
-         else
-           no_src_str = NO_DISASSEM_STRING;
-         if (strlen (no_src_str) >= half_width)
-           x_pos = 1;
-         else
-           x_pos = half_width - strlen (no_src_str);
-         mvwaddstr (win_info->handle,
-                    (win_info->height / 2),
-                    x_pos,
-                    (char *) no_src_str);
-
-         /* elz: Added this function call to set the real contents of
-            the window to what is on the screen, so that later calls
-            to refresh, do display the correct stuff, and not the old
-            image.  */
-
-         tui_set_source_content_nil (win_info, no_src_str);
-       }
+
+      const char *no_src_str;
+
+      if (win_info->type == SRC_WIN)
+       no_src_str = NO_SRC_STRING;
+      else
+       no_src_str = NO_DISASSEM_STRING;
+      if (strlen (no_src_str) >= half_width)
+       x_pos = 1;
+      else
+       x_pos = half_width - strlen (no_src_str);
+      mvwaddstr (win_info->handle,
+                (win_info->height / 2),
+                x_pos,
+                (char *) no_src_str);
+
+      /* elz: Added this function call to set the real contents of
+        the window to what is on the screen, so that later calls
+        to refresh, do display the correct stuff, and not the old
+        image.  */
+
+      tui_set_source_content_nil (win_info, no_src_str);
+
       win_info->refresh_window ();
     }
 }
@@ -266,19 +264,19 @@ tui_erase_source_content (struct tui_win_info *win_info,
 
 /* Redraw the complete line of a source or disassembly window.  */
 static void
-tui_show_source_line (struct tui_win_info *win_info, int lineno)
+tui_show_source_line (struct tui_source_window_base *win_info, int lineno)
 {
-  struct tui_win_element *line;
+  struct tui_source_element *line;
   int x;
 
-  line = win_info->content[lineno - 1];
-  if (line->which_element.source.is_exec_point)
+  line = &win_info->content[lineno - 1];
+  if (line->is_exec_point)
     tui_set_reverse_mode (win_info->handle, true);
 
   wmove (win_info->handle, lineno, 1);
-  tui_puts (line->which_element.source.line,
+  tui_puts (line->line,
            win_info->handle);
-  if (line->which_element.source.is_exec_point)
+  if (line->is_exec_point)
     tui_set_reverse_mode (win_info->handle, false);
 
   /* Clear to end of line but stop before the border.  */
@@ -291,21 +289,21 @@ tui_show_source_line (struct tui_win_info *win_info, int lineno)
 }
 
 void
-tui_show_source_content (struct tui_win_info *win_info)
+tui_show_source_content (struct tui_source_window_base *win_info)
 {
-  if (win_info->content_size > 0)
+  if (!win_info->content.empty ())
     {
       int lineno;
 
-      for (lineno = 1; lineno <= win_info->content_size; lineno++)
+      for (lineno = 1; lineno <= win_info->content.size (); lineno++)
         tui_show_source_line (win_info, lineno);
     }
   else
-    tui_erase_source_content (win_info, TRUE);
+    tui_erase_source_content (win_info);
 
   tui_check_and_display_highlight_if_needed (win_info);
   win_info->refresh_window ();
-  win_info->content_in_use = TRUE;
+  win_info->content_in_use = true;
 }
 
 /* See tui-data.h.  */
@@ -324,8 +322,7 @@ tui_source_window_base::refill ()
     }
 
   tui_update_source_window_as_is (this, gdbarch, s,
-                                 content[0]
-                                   ->which_element.source.line_or_addr,
+                                 content[0].line_or_addr,
                                  FALSE);
 }
 
@@ -334,7 +331,7 @@ tui_source_window_base::refill ()
 void
 tui_source_window_base::do_scroll_horizontal (int num_to_scroll)
 {
-  if (content != NULL)
+  if (!content.empty ())
     {
       int offset = horizontal_offset + num_to_scroll;
       if (offset < 0)
@@ -345,35 +342,35 @@ tui_source_window_base::do_scroll_horizontal (int num_to_scroll)
 }
 
 
-/* Set or clear the has_break flag in the line whose line is
+/* Set or clear the is_exec_point flag in the line whose line is
    line_no.  */
 
 void
 tui_source_window_base::set_is_exec_point_at (struct tui_line_or_address l)
 {
-  int changed = 0;
+  bool changed = false;
   int i;
 
   i = 0;
-  while (i < content_size)
+  while (i < content.size ())
     {
-      int new_state;
+      bool new_state;
       struct tui_line_or_address content_loa =
-       content[i]->which_element.source.line_or_addr;
+       content[i].line_or_addr;
 
       gdb_assert (l.loa == LOA_ADDRESS || l.loa == LOA_LINE);
       gdb_assert (content_loa.loa == LOA_LINE
                  || content_loa.loa == LOA_ADDRESS);
       if (content_loa.loa == l.loa
          && ((l.loa == LOA_LINE && content_loa.u.line_no == l.u.line_no)
-              || (content_loa.u.addr == l.u.addr)))
-        new_state = TRUE;
+              || (l.loa == LOA_ADDRESS && content_loa.u.addr == l.u.addr)))
+        new_state = true;
       else
-       new_state = FALSE;
-      if (new_state != content[i]->which_element.source.is_exec_point)
+       new_state = false;
+      if (new_state != content[i].is_exec_point)
         {
-          changed++;
-          content[i]->which_element.source.is_exec_point = new_state;
+          changed = true;
+          content[i].is_exec_point = new_state;
           tui_show_source_line (this, i + 1);
         }
       i++;
@@ -382,15 +379,14 @@ tui_source_window_base::set_is_exec_point_at (struct tui_line_or_address l)
     refill ();
 }
 
-/* Update the execution windows to show the active breakpoints.
-   This is called whenever a breakpoint is inserted, removed or
-   has its state changed.  */
+/* See tui-winsource.h.  */
+
 void
-tui_update_all_breakpoint_info ()
+tui_update_all_breakpoint_info (struct breakpoint *being_deleted)
 {
   for (tui_source_window_base *win : tui_source_windows ())
     {
-      if (tui_update_breakpoint_info (win, FALSE))
+      if (tui_update_breakpoint_info (win, being_deleted, false))
         {
           tui_update_exec_info (win);
         }
@@ -398,35 +394,34 @@ tui_update_all_breakpoint_info ()
 }
 
 
-/* Scan the source window and the breakpoints to update the has_break
+/* Scan the source window and the breakpoints to update the break_mode
    information for each line.
 
-   Returns 1 if something changed and the execution window must be
+   Returns true if something changed and the execution window must be
    refreshed.  */
 
-int
-tui_update_breakpoint_info (struct tui_win_info *win, 
-                           int current_only)
+bool
+tui_update_breakpoint_info (struct tui_source_window_base *win,
+                           struct breakpoint *being_deleted,
+                           bool current_only)
 {
   int i;
-  int need_refresh = 0;
-  tui_source_window_base *src = (tui_source_window_base *) win;
+  bool need_refresh = false;
 
-  for (i = 0; i < win->content_size; i++)
+  for (i = 0; i < win->content.size (); i++)
     {
       struct breakpoint *bp;
       extern struct breakpoint *breakpoint_chain;
-      int mode;
       struct tui_source_element *line;
 
-      line = &win->content[i]->which_element.source;
+      line = &win->content[i];
       if (current_only && !line->is_exec_point)
          continue;
 
       /* Scan each breakpoint to see if the current line has something to
          do with it.  Identify enable/disabled breakpoints as well as
          those that we already hit.  */
-      mode = 0;
+      tui_bp_flags mode = 0;
       for (bp = breakpoint_chain;
            bp != NULL;
            bp = bp->next)
@@ -436,17 +431,12 @@ tui_update_breakpoint_info (struct tui_win_info *win,
          gdb_assert (line->line_or_addr.loa == LOA_LINE
                      || line->line_or_addr.loa == LOA_ADDRESS);
 
+         if (bp == being_deleted)
+           continue;
+
          for (loc = bp->loc; loc != NULL; loc = loc->next)
            {
-             if ((win == TUI_SRC_WIN
-                  && loc->symtab != NULL
-                  && filename_cmp (src->fullname,
-                                   symtab_to_fullname (loc->symtab)) == 0
-                  && line->line_or_addr.loa == LOA_LINE
-                  && loc->line_number == line->line_or_addr.u.line_no)
-                 || (win == TUI_DISASM_WIN
-                     && line->line_or_addr.loa == LOA_ADDRESS
-                     && loc->address == line->line_or_addr.u.addr))
+             if (win->location_matches_p (loc, i))
                {
                  if (bp->enable_state == bp_disabled)
                    mode |= TUI_BP_DISABLED;
@@ -461,10 +451,10 @@ tui_update_breakpoint_info (struct tui_win_info *win,
                }
            }
         }
-      if (line->has_break != mode)
+      if (line->break_mode != mode)
         {
-          line->has_break = mode;
-          need_refresh = 1;
+          line->break_mode = mode;
+          need_refresh = true;
         }
     }
   return need_refresh;
@@ -492,21 +482,21 @@ tui_set_exec_info_content (struct tui_source_window_base *win_info)
       tui_exec_info_content *content
        = win_info->execution_info->maybe_allocate_content (win_info->height);
 
-      tui_update_breakpoint_info (win_info, 1);
-      for (int i = 0; i < win_info->content_size; i++)
+      tui_update_breakpoint_info (win_info, nullptr, true);
+      for (int i = 0; i < win_info->content.size (); i++)
        {
          tui_exec_info_content &element = content[i];
-         struct tui_win_element *src_element;
-         int mode;
+         struct tui_source_element *src_element;
+         tui_bp_flags mode;
 
-         src_element = win_info->content[i];
+         src_element = &win_info->content[i];
 
          memset (element, ' ', sizeof (tui_exec_info_content));
          element[TUI_EXECINFO_SIZE - 1] = 0;
 
          /* Now update the exec info content based upon the state
             of each line as indicated by the source content.  */
-         mode = src_element->which_element.source.has_break;
+         mode = src_element->break_mode;
          if (mode & TUI_BP_HIT)
            element[TUI_BP_HIT_POS] = (mode & TUI_BP_HARDWARE) ? 'H' : 'B';
          else if (mode & (TUI_BP_ENABLED | TUI_BP_DISABLED))
@@ -517,7 +507,7 @@ tui_set_exec_info_content (struct tui_source_window_base *win_info)
          else if (mode & TUI_BP_DISABLED)
            element[TUI_BP_BREAK_POS] = '-';
 
-         if (src_element->which_element.source.is_exec_point)
+         if (src_element->is_exec_point)
            element[TUI_EXEC_POS] = '>';
        }
     }
@@ -532,13 +522,12 @@ tui_show_exec_info_content (struct tui_source_window_base *win_info)
 
   werase (exec_info->handle);
   exec_info->refresh_window ();
-  for (int cur_line = 1; (cur_line <= win_info->content_size); cur_line++)
+  for (int cur_line = 1; cur_line <= win_info->content.size (); cur_line++)
     mvwaddstr (exec_info->handle,
               cur_line,
               0,
               content[cur_line - 1]);
   exec_info->refresh_window ();
-  exec_info->content_in_use = TRUE;
 }
 
 
@@ -554,7 +543,6 @@ tui_erase_exec_info_content (struct tui_source_window_base *win_info)
 void
 tui_clear_exec_info_content (struct tui_source_window_base *win_info)
 {
-  win_info->execution_info->content_in_use = FALSE;
   tui_erase_exec_info_content (win_info);
 }
 
@@ -567,7 +555,7 @@ tui_update_exec_info (struct tui_source_window_base *win_info)
 }
 
 void
-tui_alloc_source_buffer (struct tui_win_info *win_info)
+tui_alloc_source_buffer (struct tui_source_window_base *win_info)
 {
   int i, line_width, max_lines;
 
@@ -577,13 +565,11 @@ tui_alloc_source_buffer (struct tui_win_info *win_info)
   line_width = win_info->width - 2 + 1;
 
   /* Allocate the buffer for the source lines.  */
-  if (win_info->content == NULL)
+  win_info->content.resize (max_lines);
+  for (i = 0; i < max_lines; i++)
     {
-      /* Allocate the content list.  */
-      win_info->content = tui_alloc_content (max_lines, SRC_WIN);
-      for (i = 0; i < max_lines; i++)
-       win_info->content[i]->which_element.source.line
-         = (char *) xmalloc (line_width);
+      if (win_info->content[i].line == nullptr)
+       win_info->content[i].line = (char *) xmalloc (line_width);
     }
 }
 
@@ -592,7 +578,7 @@ tui_alloc_source_buffer (struct tui_win_info *win_info)
    in the current source window.  */
 int
 tui_line_is_displayed (int line, 
-                      struct tui_win_info *win_info,
+                      struct tui_source_window_base *win_info,
                       int check_threshold)
 {
   int is_displayed = FALSE;
@@ -603,14 +589,12 @@ tui_line_is_displayed (int line,
   else
     threshold = 0;
   i = 0;
-  while (i < win_info->content_size - threshold
+  while (i < win_info->content.size () - threshold
         && !is_displayed)
     {
       is_displayed
-       = win_info->content[i]
-           ->which_element.source.line_or_addr.loa == LOA_LINE
-         && win_info->content[i]
-              ->which_element.source.line_or_addr.u.line_no == line;
+       = win_info->content[i].line_or_addr.loa == LOA_LINE
+         && win_info->content[i].line_or_addr.u.line_no == line;
       i++;
     }
 
@@ -622,7 +606,7 @@ tui_line_is_displayed (int line,
    in the current source window.  */
 int
 tui_addr_is_displayed (CORE_ADDR addr, 
-                      struct tui_win_info *win_info,
+                      struct tui_source_window_base *win_info,
                       int check_threshold)
 {
   int is_displayed = FALSE;
@@ -633,14 +617,12 @@ tui_addr_is_displayed (CORE_ADDR addr,
   else
     threshold = 0;
   i = 0;
-  while (i < win_info->content_size - threshold
+  while (i < win_info->content.size () - threshold
         && !is_displayed)
     {
       is_displayed
-       = win_info->content[i]
-           ->which_element.source.line_or_addr.loa == LOA_ADDRESS
-         && win_info->content[i]
-              ->which_element.source.line_or_addr.u.addr == addr;
+       = win_info->content[i].line_or_addr.loa == LOA_ADDRESS
+         && win_info->content[i].line_or_addr.u.addr == addr;
       i++;
     }
 
This page took 0.034185 seconds and 4 git commands to generate.