* frame.c (frame_pop, frame_observer_target_changed): Call

	reinit_frame_cache.
	(flush_cached_frames): Rename to reinit_frame_cache and delete
	old implementation.
	* frame.h (flush_cached_frames): Delete prototype and update comment.

	* bsd-kvm.c (bsd_kvm_open, bsd_kvm_proc_cmd, bsd_kvm_pcb_cmd): Call
	reinit_frame_cache instead of flush_cached_frames.  Do not call
	select_frame after reinit_frame_cache.
	* corelow.c (core_open): Likewise.
	* gdbarch.sh (deprecated_current_gdbarch_select_hack): Likewise.
	* infrun.c (prepare_to_proceed, context_switch)
	(handle_inferior_event): Likewise.
	* linux-fork.c (fork_load_infrun_state): Likewise.
	* ocd.c (ocd_start_remote): Likewise.
	* remote-e7000.c (e7000_start_remote): Likewise.
	* remote-mips.c (device): Likewise.
	* thread.c (switch_to_thread): Likewise.
	* tracepoint.c (finish_tfind_command): Likewise.
	* gdbarch.c: Regenerated.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 096d9be57c4cf359fba9f4c8d976e9cf77f3cb73..250bc494b822b459044cf47f9f4407be37670fb8 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,26 @@
+2007-02-28  Daniel Jacobowitz  <dan@codesourcery.com>
+
+       * frame.c (frame_pop, frame_observer_target_changed): Call
+       reinit_frame_cache.
+       (flush_cached_frames): Rename to reinit_frame_cache and delete
+       old implementation.
+       * frame.h (flush_cached_frames): Delete prototype and update comment.
+
+       * bsd-kvm.c (bsd_kvm_open, bsd_kvm_proc_cmd, bsd_kvm_pcb_cmd): Call
+       reinit_frame_cache instead of flush_cached_frames.  Do not call
+       select_frame after reinit_frame_cache.
+       * corelow.c (core_open): Likewise.
+       * gdbarch.sh (deprecated_current_gdbarch_select_hack): Likewise.
+       * infrun.c (prepare_to_proceed, context_switch)
+       (handle_inferior_event): Likewise.
+       * linux-fork.c (fork_load_infrun_state): Likewise.
+       * ocd.c (ocd_start_remote): Likewise.
+       * remote-e7000.c (e7000_start_remote): Likewise.
+       * remote-mips.c (device): Likewise.
+       * thread.c (switch_to_thread): Likewise.
+       * tracepoint.c (finish_tfind_command): Likewise.
+       * gdbarch.c: Regenerated.
+
 2007-02-28  Jerome Guitton  <guitton@adacore.com>
             Joel Brobecker  <brobecker@adacore.com>
 

diff --git a/gdb/bsd-kvm.c b/gdb/bsd-kvm.c
index a60204332a61af6386d6d3e9ca79e80cd062f3a1..864be4f45171cca6daccf39757a49a8a3327cc88 100644 (file)
--- a/gdb/bsd-kvm.c
+++ b/gdb/bsd-kvm.c
@@ -93,8 +93,7 @@ bsd_kvm_open (char *filename, int from_tty)
 
   target_fetch_registers (-1);
 
-  flush_cached_frames ();
-  select_frame (get_current_frame ());
+  reinit_frame_cache ();
   print_stack_frame (get_selected_frame (NULL), -1, 1);
 }
 
@@ -276,8 +275,7 @@ bsd_kvm_proc_cmd (char *arg, int fromtty)
 
   target_fetch_registers (-1);
 
-  flush_cached_frames ();
-  select_frame (get_current_frame ());
+  reinit_frame_cache ();
   print_stack_frame (get_selected_frame (NULL), -1, 1);
 }
 
@@ -297,8 +295,7 @@ bsd_kvm_pcb_cmd (char *arg, int fromtty)
 
   target_fetch_registers (-1);
 
-  flush_cached_frames ();
-  select_frame (get_current_frame ());
+  reinit_frame_cache ();
   print_stack_frame (get_selected_frame (NULL), -1, 1);
 }
 

diff --git a/gdb/corelow.c b/gdb/corelow.c
index 586ac69afe0571f8b192d03c46a99b263db6c5a9..267833e7b2b0150953f952d610d880f5fa60b73f 100644 (file)
--- a/gdb/corelow.c
+++ b/gdb/corelow.c
@@ -378,8 +378,7 @@ core_open (char *filename, int from_tty)
       target_fetch_registers (-1);
 
       /* Now, set up the frame cache, and print the top of stack.  */
-      flush_cached_frames ();
-      select_frame (get_current_frame ());
+      reinit_frame_cache ();
       print_stack_frame (get_selected_frame (NULL), 1, SRC_AND_LOC);
     }
   else

diff --git a/gdb/frame.c b/gdb/frame.c
index ff8078bf26d08a475b015bc4dafe16ac993bb9d9..a40bbb219c31f2b6dfd73dad0e908dac7256e32e 100644 (file)
--- a/gdb/frame.c
+++ b/gdb/frame.c
@@ -537,7 +537,7 @@ frame_pop (struct frame_info *this_frame)
 
   /* We've made right mess of GDB's local state, just discard
      everything.  */
-  flush_cached_frames ();
+  reinit_frame_cache ();
 }
 
 void
@@ -1070,13 +1070,13 @@ get_next_frame (struct frame_info *this_frame)
 void
 frame_observer_target_changed (struct target_ops *target)
 {
-  flush_cached_frames ();
+  reinit_frame_cache ();
 }
 
 /* Flush the entire frame cache.  */
 
 void
-flush_cached_frames (void)
+reinit_frame_cache (void)
 {
   /* Since we can't really be sure what the first object allocated was */
   obstack_free (&frame_cache_obstack, 0);
@@ -1086,21 +1086,7 @@ flush_cached_frames (void)
   select_frame (NULL);
   annotate_frames_invalid ();
   if (frame_debug)
-    fprintf_unfiltered (gdb_stdlog, "{ flush_cached_frames () }\n");
-}
-
-/* Flush the frame cache, and start a new one if necessary.  */
-
-void
-reinit_frame_cache (void)
-{
-  flush_cached_frames ();
-
-  /* FIXME: The inferior_ptid test is wrong if there is a corefile.  */
-  if (PIDGET (inferior_ptid) != 0)
-    {
-      select_frame (get_current_frame ());
-    }
+    fprintf_unfiltered (gdb_stdlog, "{ reinit_frame_cache () }\n");
 }
 
 /* Find where a register is saved (in memory or another register).

diff --git a/gdb/frame.h b/gdb/frame.h
index 941ee73ef5b5ae292ac61f41fe597f981a7af7f5..5181855564c56463937441321c9732f5f4c6af41 100644 (file)
--- a/gdb/frame.h
+++ b/gdb/frame.h
@@ -225,17 +225,10 @@ extern struct frame_info *get_current_frame (void);
 /* Invalidates the frame cache (this function should have been called
    invalidate_cached_frames).
 
-   FIXME: cagney/2002-11-28: The only difference between
-   flush_cached_frames() and reinit_frame_cache() is that the latter
-   explicitly sets the selected frame back to the current frame -- there
-   isn't any real difference (except that one delays the selection of
-   a new frame).  Code can instead simply rely on get_selected_frame()
-   to reinit the selected frame as needed.  As for invalidating the
-   cache, there should be two methods: one that reverts the thread's
-   selected frame back to current frame (for when the inferior
-   resumes) and one that does not (for when the user modifies the
-   target invalidating the frame cache).  */
-extern void flush_cached_frames (void);
+   FIXME: cagney/2002-11-28: There should be two methods: one that
+   reverts the thread's selected frame back to current frame (for when
+   the inferior resumes) and one that does not (for when the user
+   modifies the target invalidating the frame cache).  */
 extern void reinit_frame_cache (void);
 
 /* On demand, create the selected frame and then return it.  If the

diff --git a/gdb/gdbarch.c b/gdb/gdbarch.c
index 562897a72cf85c3a8c0424ff3d2208086c1dba3f..99f6ddf5fa6af1573b012a9dbbc1621166dde4e3 100644 (file)
--- a/gdb/gdbarch.c
+++ b/gdb/gdbarch.c
@@ -4202,7 +4202,7 @@ deprecated_current_gdbarch_select_hack (struct gdbarch *new_gdbarch)
   current_gdbarch_swap_out_hack ();
   current_gdbarch_swap_in_hack (new_gdbarch);
   architecture_changed_event ();
-  flush_cached_frames ();
+  reinit_frame_cache ();
 }
 
 extern void _initialize_gdbarch (void);

diff --git a/gdb/gdbarch.sh b/gdb/gdbarch.sh
index c776e28ad99c58852595c32b9c6d0fd486f43091..e65716cfb54426712a7255cac7bb82d1c2f3a561 100755 (executable)
--- a/gdb/gdbarch.sh
+++ b/gdb/gdbarch.sh
@@ -2241,7 +2241,7 @@ deprecated_current_gdbarch_select_hack (struct gdbarch *new_gdbarch)
   current_gdbarch_swap_out_hack ();
   current_gdbarch_swap_in_hack (new_gdbarch);
   architecture_changed_event ();
-  flush_cached_frames ();
+  reinit_frame_cache ();
 }
 
 extern void _initialize_gdbarch (void);

diff --git a/gdb/infrun.c b/gdb/infrun.c
index 09467dbf5267bc174ad8aad604a0798543705acb..5d03619930f3666fc1d6c0acd630b86eb0c56da3 100644 (file)
--- a/gdb/infrun.c
+++ b/gdb/infrun.c
@@ -686,10 +686,9 @@ prepare_to_proceed (void)
 
          /* FIXME: This stuff came from switch_to_thread() in
             thread.c (which should probably be a public function).  */
-         flush_cached_frames ();
+         reinit_frame_cache ();
          registers_changed ();
          stop_pc = wait_pc;
-         select_frame (get_current_frame ());
        }
 
       /* We return 1 to indicate that there is a breakpoint here,
@@ -1157,7 +1156,7 @@ context_switch (struct execution_control_state *ecs)
                         &ecs->current_line, &ecs->current_symtab);
     }
   inferior_ptid = ecs->ptid;
-  flush_cached_frames ();
+  reinit_frame_cache ();
 }
 
 static void
@@ -1304,7 +1303,7 @@ handle_inferior_event (struct execution_control_state *ecs)
     }
   ecs->infwait_state = infwait_normal_state;
 
-  flush_cached_frames ();
+  reinit_frame_cache ();
 
   /* If it's a new process, add it to the thread database */
 
@@ -1429,7 +1428,7 @@ handle_inferior_event (struct execution_control_state *ecs)
       if (!ptid_equal (ecs->ptid, inferior_ptid))
        {
          context_switch (ecs);
-         flush_cached_frames ();
+         reinit_frame_cache ();
        }
 
       stop_pc = read_pc ();
@@ -1490,7 +1489,7 @@ handle_inferior_event (struct execution_control_state *ecs)
       if (!ptid_equal (ecs->ptid, inferior_ptid))
        {
          context_switch (ecs);
-         flush_cached_frames ();
+         reinit_frame_cache ();
        }
 
       /* If no catchpoint triggered for this, then keep going.  */

diff --git a/gdb/linux-fork.c b/gdb/linux-fork.c
index 6771784c34bd56384ce93987142fe42e3a5cdd4e..0da442a993a40b98a578028da1880ac80c660740 100644 (file)
--- a/gdb/linux-fork.c
+++ b/gdb/linux-fork.c
@@ -250,10 +250,6 @@ fork_load_infrun_state (struct fork_info *fp)
   registers_changed ();
   reinit_frame_cache ();
 
-  /* We must select a new frame before making any inferior calls to
-     avoid warnings.  */
-  select_frame (get_current_frame ());
-
   stop_pc = read_pc ();
   nullify_last_target_wait_ptid ();
 

diff --git a/gdb/ocd.c b/gdb/ocd.c
index 5e72b10fd630dcbeefb851af78a21f50f0a6385c..f52990088aac75d21c319941edc04b6f7ac05ff7 100644 (file)
--- a/gdb/ocd.c
+++ b/gdb/ocd.c
@@ -223,7 +223,7 @@ ocd_start_remote (void *dummy)
    doesn't happen here (in fact, it may not be possible to get the monitor to
    send the appropriate packet).  */
 
-  flush_cached_frames ();
+  reinit_frame_cache ();
   registers_changed ();
   stop_pc = read_pc ();
   print_stack_frame (get_selected_frame (NULL), 0, SRC_AND_LOC);

diff --git a/gdb/remote-e7000.c b/gdb/remote-e7000.c
index 4d94347f09bfe675859436779c5455596793d948..fea809df0266f17df6948b8ef7ad62c05548cfb9 100644 (file)
--- a/gdb/remote-e7000.c
+++ b/gdb/remote-e7000.c
@@ -628,7 +628,7 @@ e7000_start_remote (void *dummy)
    that the target is about to print out a status message of some sort.  That
    doesn't happen here. */
 
-  flush_cached_frames ();
+  reinit_frame_cache ();
   registers_changed ();
   stop_pc = read_pc ();
   print_stack_frame (get_selected_frame (NULL), 0, SRC_AND_LOC);

diff --git a/gdb/remote-mips.c b/gdb/remote-mips.c
index 4c6fdf8ea1445c01c6afc3bb223dc80db48ab6d9..1f92f999f93cb7b7981c432b5f56630dded6f0a9 100644 (file)
--- a/gdb/remote-mips.c
+++ b/gdb/remote-mips.c
@@ -1584,7 +1584,7 @@ device is attached to the target board (e.g., /dev/ttya).\n"
      of some sort.  That doesn't happen here (in fact, it may not be
      possible to get the monitor to send the appropriate packet).  */
 
-  flush_cached_frames ();
+  reinit_frame_cache ();
   registers_changed ();
   stop_pc = read_pc ();
   print_stack_frame (get_selected_frame (NULL), 0, SRC_AND_LOC);

diff --git a/gdb/thread.c b/gdb/thread.c
index a7a8617e380f130435f58c44e00ceac9ff29b813..3281988579f2a5d6769effb87529668dd5fc3865 100644 (file)
--- a/gdb/thread.c
+++ b/gdb/thread.c
@@ -461,10 +461,9 @@ switch_to_thread (ptid_t ptid)
     return;
 
   inferior_ptid = ptid;
-  flush_cached_frames ();
+  reinit_frame_cache ();
   registers_changed ();
   stop_pc = read_pc ();
-  select_frame (get_current_frame ());
 }
 
 static void

diff --git a/gdb/tracepoint.c b/gdb/tracepoint.c
index 3ad005f1cabf3bd713948f9b08f4cd0763479465..fb771d2223be7faa8a4c8e1d17deff2699f65aad 100644 (file)
--- a/gdb/tracepoint.c
+++ b/gdb/tracepoint.c
@@ -1969,9 +1969,8 @@ finish_tfind_command (char **msg,
        error (_("Bogus reply from target: %s"), reply);
       }
 
-  flush_cached_frames ();
+  reinit_frame_cache ();
   registers_changed ();
-  select_frame (get_current_frame ());
   set_traceframe_num (target_frameno);
   set_tracepoint_num (target_tracept);
   if (target_frameno == -1)