Move local variables to inner block
authorYao Qi <yao@codesourcery.com>
Tue, 24 Jun 2014 13:47:14 +0000 (21:47 +0800)
committerYao Qi <yao@codesourcery.com>
Thu, 26 Jun 2014 08:26:01 +0000 (16:26 +0800)
dummy_frame_sniffer has two local variables dummyframe and this_id,
but they are only used in the if block below.  This patch is to move
them into the inner block.

gdb:

2014-06-26  Yao Qi  <yao@codesourcery.com>

* dummy-frame.c (dummy_frame_sniffer): Move local variables
dummyframe and this_id into inner block below.

gdb/ChangeLog
gdb/dummy-frame.c

index 66484b5..2b7e33c 100644 (file)
@@ -1,5 +1,10 @@
 2014-06-26  Yao Qi  <yao@codesourcery.com>
 
+       * dummy-frame.c (dummy_frame_sniffer): Move local variables
+       dummyframe and this_id into inner block below.
+
+2014-06-26  Yao Qi  <yao@codesourcery.com>
+
        * infrun.c (_initialize_infrun): Replace "signal_program[0]"
        with "signal_pass[0]" in the initialization of signal_pass.
 
index 998ca93..dd2ff12 100644 (file)
@@ -195,9 +195,6 @@ dummy_frame_sniffer (const struct frame_unwind *self,
                     struct frame_info *this_frame,
                     void **this_prologue_cache)
 {
-  struct dummy_frame *dummyframe;
-  struct frame_id this_id;
-
   /* When unwinding a normal frame, the stack structure is determined
      by analyzing the frame's function's code (be it using brute force
      prologue analysis, or the dwarf2 CFI).  In the case of a dummy
@@ -209,9 +206,11 @@ dummy_frame_sniffer (const struct frame_unwind *self,
   /* Don't bother unless there is at least one dummy frame.  */
   if (dummy_frame_stack != NULL)
     {
+      struct dummy_frame *dummyframe;
       /* Use an architecture specific method to extract this frame's
         dummy ID, assuming it is a dummy frame.  */
-      this_id = gdbarch_dummy_id (get_frame_arch (this_frame), this_frame);
+      struct frame_id this_id
+       = gdbarch_dummy_id (get_frame_arch (this_frame), this_frame);
 
       /* Use that ID to find the corresponding cache entry.  */
       for (dummyframe = dummy_frame_stack;