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 66484b599ab708005d05be06336c8dbb247abece..2b7e33ca3a97d869a7f87c0a9346e81be02d9ff8 100644 (file)
@@ -1,3 +1,8 @@
+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]"
index 998ca934805308ebe9c4a5eef0730190101d7f62..dd2ff1285ce7f7fbe152e80592dbda68cdd791d3 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;