From 66076460436b4f9f7917fce7b073fcc79cb81ce9 Mon Sep 17 00:00:00 2001 From: Daniel Jacobowitz Date: Fri, 13 Nov 2009 22:05:06 +0000 Subject: [PATCH] 2009-11-13 Maciej W. Rozycki * breakpoint.c (update_watchpoint): Only fiddle with frames for local watchpoints. --- gdb/ChangeLog | 5 +++++ gdb/breakpoint.c | 20 +++++++++++++------- 2 files changed, 18 insertions(+), 7 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index cf4d2c0c532..e55b4d36061 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,8 @@ +2009-11-13 Maciej W. Rozycki + + * breakpoint.c (update_watchpoint): Only fiddle with frames for + local watchpoints. + 2009-11-13 Pedro Alves * infrun.c (handle_inferior_event): Set stop_print_frame on diff --git a/gdb/breakpoint.c b/gdb/breakpoint.c index 757c76d424d..cadc5364596 100644 --- a/gdb/breakpoint.c +++ b/gdb/breakpoint.c @@ -1000,6 +1000,7 @@ update_watchpoint (struct breakpoint *b, int reparse) int within_current_scope; struct frame_id saved_frame_id; struct bp_location *loc; + int frame_saved; bpstat bs; struct program_space *frame_pspace; @@ -1011,12 +1012,7 @@ update_watchpoint (struct breakpoint *b, int reparse) if (b->disposition == disp_del_at_next_stop) return; - /* Save the current frame's ID so we can restore it after - evaluating the watchpoint expression on its own frame. */ - /* FIXME drow/2003-09-09: It would be nice if evaluate_expression - took a frame parameter, so that we didn't have to change the - selected frame. */ - saved_frame_id = get_frame_id (get_selected_frame (NULL)); + frame_saved = 0; /* Determine if the watchpoint is within scope. */ if (b->exp_valid_block == NULL) @@ -1024,6 +1020,15 @@ update_watchpoint (struct breakpoint *b, int reparse) else { struct frame_info *fi; + + /* Save the current frame's ID so we can restore it after + evaluating the watchpoint expression on its own frame. */ + /* FIXME drow/2003-09-09: It would be nice if evaluate_expression + took a frame parameter, so that we didn't have to change the + selected frame. */ + frame_saved = 1; + saved_frame_id = get_frame_id (get_selected_frame (NULL)); + fi = frame_find_by_id (b->watchpoint_frame); within_current_scope = (fi != NULL); if (within_current_scope) @@ -1169,7 +1174,8 @@ in which its expression is valid.\n"), } /* Restore the selected frame. */ - select_frame (frame_find_by_id (saved_frame_id)); + if (frame_saved) + select_frame (frame_find_by_id (saved_frame_id)); } -- 2.30.2