diff options
| author | Pedro Alves <palves@redhat.com> | 2013-08-30 15:32:45 +0000 |
|---|---|---|
| committer | Pedro Alves <palves@redhat.com> | 2013-08-30 15:32:45 +0000 |
| commit | 0684eb99279cdc08aefe1ef9876f890b9bf72659 (patch) | |
| tree | ce7efe9d93df9e3aaac6c0d13834acbfb84db108 /gdb/thread.c | |
| parent | 80423d1a0369e89da499ef1df3bfe38c83bd795e (diff) | |
| download | gdb-0684eb99279cdc08aefe1ef9876f890b9bf72659.tar.gz | |
restore_selected_frame: tweak warning.
I noticed SRC_LINE has special handling within print_stack_frame (mid
statement handling), so I audited all uses, and noticed the one in
restore_selected_frame. I actually added this warning myself back in
2008, but reading back, I think we can do better. "reparsed frame" is
probably confusing to users.
Old:
warning: Couldn't restore frame #2 in current thread, at reparsed frame #0
45 w = 0;
(gdb)
New:
warning: Couldn't restore frame #2 in current thread. Bottom (innermost) frame selected:
#0 foo () at foo.c:45
45 w = 0;
(gdb)
Tested on x86_64 Fedora 17.
gdb/
2013-08-30 Pedro Alves <palves@redhat.com>
* thread.c (restore_selected_frame): Use SRC_AND_LOC, and change
warning text.
Diffstat (limited to 'gdb/thread.c')
| -rw-r--r-- | gdb/thread.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/gdb/thread.c b/gdb/thread.c index 78851e462d0..52626ff0989 100644 --- a/gdb/thread.c +++ b/gdb/thread.c @@ -1077,12 +1077,12 @@ restore_selected_frame (struct frame_id a_frame_id, int frame_level) if (frame_level > 0 && !ui_out_is_mi_like_p (current_uiout)) { warning (_("Couldn't restore frame #%d in " - "current thread, at reparsed frame #0\n"), + "current thread. Bottom (innermost) frame selected:"), frame_level); /* For MI, we should probably have a notification about current frame change. But this error is not very likely, so don't bother for now. */ - print_stack_frame (get_selected_frame (NULL), 1, SRC_LINE); + print_stack_frame (get_selected_frame (NULL), 1, SRC_AND_LOC); } } |