gdb/
authorPedro Alves <palves@redhat.com>
Fri, 18 Mar 2011 18:50:20 +0000 (18:50 +0000)
committerPedro Alves <palves@redhat.com>
Fri, 18 Mar 2011 18:50:20 +0000 (18:50 +0000)
* tui/tui-hooks.c (tui_selected_frame_level_changed_hook): Assume
there's always a frame.  Use get_frame_pc_if_available instead of
get_frame_pc, and if there's no PC available, don't look up a
symtab.

gdb/ChangeLog
gdb/tui/tui-hooks.c

index 7fb05100cf4f3740d82c4de2f8812295a1fb3bc0..0dbc501d99ef8395d562b339060b24404234e322 100644 (file)
@@ -1,3 +1,10 @@
+2011-03-18  Pedro Alves  <pedro@codesourcery.com>
+
+       * tui/tui-hooks.c (tui_selected_frame_level_changed_hook): Assume
+       there's always a frame.  Use get_frame_pc_if_available instead of
+       get_frame_pc, and if there's no PC available, don't look up a
+       symtab.
+
 2011-03-18  Pedro Alves  <pedro@codesourcery.com>
 
        * stack.c (print_frame_local_vars, print_frame_arg_vars): Handle
index e132b5a1b3d5cb190b455595267d9cf9f0a2ff39..97b99a786a7d41257944ba632c0574854c1ba665 100644 (file)
@@ -190,6 +190,7 @@ static void
 tui_selected_frame_level_changed_hook (int level)
 {
   struct frame_info *fi;
+  CORE_ADDR pc;
 
   /* Negative level means that the selected frame was cleared.  */
   if (level < 0)
@@ -199,28 +200,29 @@ tui_selected_frame_level_changed_hook (int level)
   /* Ensure that symbols for this frame are read in.  Also, determine
      the source language of this frame, and switch to it if
      desired.  */
-  if (fi)
+  if (get_frame_pc_if_available (fi, &pc))
     {
       struct symtab *s;
-      
-      s = find_pc_symtab (get_frame_pc (fi));
+
+      s = find_pc_symtab (pc);
       /* elz: This if here fixes the problem with the pc not being
-         displayed in the tui asm layout, with no debug symbols.  The
-         value of s would be 0 here, and select_source_symtab would
-         abort the command by calling the 'error' function.  */
+        displayed in the tui asm layout, with no debug symbols.  The
+        value of s would be 0 here, and select_source_symtab would
+        abort the command by calling the 'error' function.  */
       if (s)
-        select_source_symtab (s);
-
-      /* Display the frame position (even if there is no symbols).  */
-      tui_show_frame_info (fi);
-
-      /* Refresh the register window if it's visible.  */
-      if (tui_is_window_visible (DATA_WIN))
-        {
-          tui_refreshing_registers = 1;
-          tui_check_data_values (fi);
-          tui_refreshing_registers = 0;
-        }
+       select_source_symtab (s);
+    }
+
+  /* Display the frame position (even if there is no symbols or the PC
+     is not known).  */
+  tui_show_frame_info (fi);
+
+  /* Refresh the register window if it's visible.  */
+  if (tui_is_window_visible (DATA_WIN))
+    {
+      tui_refreshing_registers = 1;
+      tui_check_data_values (fi);
+      tui_refreshing_registers = 0;
     }
 }