Simplify tui_update_source_windows_with_addr
authorTom Tromey <tom@tromey.com>
Wed, 13 Nov 2019 00:59:06 +0000 (17:59 -0700)
committerTom Tromey <tom@tromey.com>
Fri, 20 Dec 2019 16:15:53 +0000 (09:15 -0700)
After the previous changes, tui_update_source_windows_with_addr simply
updates each source-like window separately, passing the same data to
each.  So, it can be simplified by using a loop instead.

gdb/ChangeLog
2019-12-20  Tom Tromey  <tom@tromey.com>

* tui/tui-winsource.c (tui_update_source_windows_with_addr):
Simplify.

Change-Id: Id2ba6b3145ec005dbed1b1115118bd1ef4efb842

gdb/ChangeLog
gdb/tui/tui-winsource.c

index feb406be6b6ed541cddadf2c0d9d63bfe724b66f..77620d046a364b762db437e91303bf5bb36ec5d9 100644 (file)
@@ -1,3 +1,8 @@
+2019-12-20  Tom Tromey  <tom@tromey.com>
+
+       * tui/tui-winsource.c (tui_update_source_windows_with_addr):
+       Simplify.
+
 2019-12-20  Tom Tromey  <tom@tromey.com>
 
        * tui/tui-winsource.h (struct tui_source_window_base)
index cad7deab804e7fd9824a4b89511454d47976276a..8f9890433738c3743dbd012ec867d7269db3a3a0 100644 (file)
@@ -206,21 +206,12 @@ tui_source_window_base::update_source_window_as_is
 void
 tui_update_source_windows_with_addr (struct gdbarch *gdbarch, CORE_ADDR addr)
 {
+  struct symtab_and_line sal {};
   if (addr != 0)
-    {
-      struct symtab_and_line sal = find_pc_line (addr, 0);
-      
-      if (TUI_DISASM_WIN != nullptr)
-       TUI_DISASM_WIN->update_source_window (gdbarch, sal);
+    sal = find_pc_line (addr, 0);
 
-      if (TUI_SRC_WIN != nullptr)
-       TUI_SRC_WIN->update_source_window (gdbarch, sal);
-    }
-  else
-    {
-      for (struct tui_source_window_base *win_info : tui_source_windows ())
-       win_info->erase_source_content ();
-    }
+  for (struct tui_source_window_base *win_info : tui_source_windows ())
+    win_info->update_source_window (gdbarch, sal);
 }
 
 /* Function to ensure that the source and/or disassemly windows