This is the mail archive of the gdb-patches@sources.redhat.com mailing list for the GDB project.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: backtrace changes current source location

Felix Lee wrote: 
ok, here's a new patch and testcase.  built and tested on
i686-pc-linux-gnu.  the testcase adds 7 PASS and 1 FAIL, which
becomes 8 PASS with the patch.

gdb/ChangeLog
2004-10-31  Felix Lee  <felix+log1@specifixinc.com>

        * stack.c (set_current_sal_from_frame): New function.
        (print_args_stub): Use it.
        (print_frame_info): Don't set sal here.

Hey, cute. Yes, thanks!

Andrew

gdb/testsuite/ChangeLog
2004-10-31  Felix Lee  <felix+log1@specifixinc.com>

        * gdb.base/cursal.exp: New file.
        * gdb.base/cursal.c: New file.

Index: gdb/stack.c
===================================================================
RCS file: /cvs/src/src/gdb/stack.c,v
retrieving revision 1.115
diff -p -u -r1.115 stack.c
--- gdb/stack.c 30 Oct 2004 21:16:10 -0000 1.115
+++ gdb/stack.c 31 Oct 2004 10:42:55 -0000
@@ -100,6 +100,8 @@ static void print_frame (struct frame_in
int print_args, struct symtab_and_line sal);
+static void set_current_sal_from_frame (struct frame_info *, int);
+
static void backtrace_command (char *, int);
static void frame_info (char *, int);
@@ -128,9 +130,12 @@ struct print_stack_frame_args
static int
print_stack_frame_stub (void *args)
{
- struct print_stack_frame_args *p = (struct print_stack_frame_args *) args;
+ struct print_stack_frame_args *p = args;
+ int center = (p->print_what == SRC_LINE
+ || p->print_what == SRC_AND_LOC);
print_frame_info (p->fi, p->print_level, p->print_what, p->print_args);
+ set_current_sal_from_frame (p->fi, center);
return 0;
}
@@ -401,6 +406,24 @@ print_args_stub (void *args)
return 0;
}
+/* Set the current source and line to the location of the given
+ frame, if possible. When CENTER is true, adjust so the
+ relevant line is in the center of the next 'list'. */
+
+static void
+set_current_sal_from_frame (struct frame_info *fi, int center)
+{
+ struct symtab_and_line sal;
+
+ find_frame_sal (fi, &sal);
+ if (sal.symtab)
+ {
+ if (center)
+ sal.line = max (sal.line - get_lines_to_list () / 2, 1);
+ set_current_source_symtab_and_line (&sal);
+ }
+}
+
/* Print information about a frame for frame "fi" at level "level".
Used in "where" output, also used to emit breakpoint or step
messages. @@ -477,12 +500,8 @@ print_frame_info (struct frame_info *fi,
source_print = (print_what == SRC_LINE || print_what == SRC_AND_LOC);
- if (sal.symtab)
- set_current_source_symtab_and_line (&sal);
-
if (source_print && sal.symtab)
{
- struct symtab_and_line cursal;
int done = 0;
int mid_statement = ((print_what == SRC_LINE)
&& (get_frame_pc (fi) != sal.pc));
@@ -513,11 +532,6 @@ print_frame_info (struct frame_info *fi,
print_source_lines (sal.symtab, sal.line, sal.line + 1, 0);
}
}
- /* Make sure we have at least a default source file */
- set_default_source_symtab_and_line ();
- cursal = get_current_source_symtab_and_line ();
- cursal.line = max (sal.line - get_lines_to_list () / 2, 1);
- set_current_source_symtab_and_line (&cursal);
}
if (print_what != LOCATION)
Index: gdb/testsuite/gdb.base/cursal.c
===================================================================
RCS file: gdb/testsuite/gdb.base/cursal.c
diff -N gdb/testsuite/gdb.base/cursal.c
--- /dev/null 1 Jan 1970 00:00:00 -0000
+++ gdb/testsuite/gdb.base/cursal.c 31 Oct 2004 10:42:55 -0000
@@ -0,0 +1,19 @@
+void
+func2 ()
+{
+ int v2 = 2;
+}
+
+void
+func1 ()
+{
+ func2 ();
+}
+
+
+int
+main ()
+{
+ int v0 = 0;
+ func1 ();
+}
Index: gdb/testsuite/gdb.base/cursal.exp
===================================================================
RCS file: gdb/testsuite/gdb.base/cursal.exp
diff -N gdb/testsuite/gdb.base/cursal.exp
--- /dev/null 1 Jan 1970 00:00:00 -0000
+++ gdb/testsuite/gdb.base/cursal.exp 31 Oct 2004 10:42:55 -0000
@@ -0,0 +1,78 @@
+# Copyright 2004 Free Software Foundation, Inc.
+
+# This program is free software; you can redistribute it and/or modify
+# it under the terms of the GNU General Public License as published by
+# the Free Software Foundation; either version 2 of the License, or
+# (at your option) any later version.
+#
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+# GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License
+# along with this program; if not, write to the Free Software
+# Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA. +
+if $tracelevel {
+ strace $tracelevel
+}
+
+set prms_id 0
+set bug_id 0
+
+set testfile "cursal"
+set srcfile ${testfile}.c
+set binfile ${objdir}/${subdir}/${testfile}
+
+if {[gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable {debug}] != ""} {
+ gdb_suppress_entire_file "Testcase compile failed, so all tests in this file will automatically fail."
+}
+
+gdb_exit
+gdb_start
+gdb_reinitialize_dir $srcdir/$subdir
+gdb_load ${binfile}
+
+gdb_test "set listsize 1" \
+ ".*"
+
+# initial sal should be first statement in main
+gdb_test "list" \
+ "v0 = 0;" \
+ "list before run"
+
+if {! [runto_main]} {
+ gdb_suppress_tests
+}
+
+gdb_test "list" \
+ "v0 = 0;" \
+ "list in main"
+
+if {! [runto "func2"]} {
+ gdb_suppress_tests
+}
+
+gdb_test "list" \
+ "v2 = 2;" \
+ "list in func2"
+
+# make sure backtrace doesn't change current source location.
+gdb_test "backtrace" \
+ ".*"
+gdb_test "list -1" \
+ "v2 = 2;" \
+ "list after backtrace"
+
+# check the window
+gdb_test "set listsize 3" \
+ ".*"
+if {! [runto_main]} {
+ gdb_suppress_tests
+}
+gdb_test "list" \
+ "func1 \\(\\);" \
+ "list size 3"
+
+return 0


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]