aboutsummaryrefslogtreecommitdiff
path: root/gdb
diff options
context:
space:
mode:
authorPedro Alves <palves@redhat.com>2013-10-28 18:34:37 +0000
committerPedro Alves <palves@redhat.com>2013-10-28 18:34:37 +0000
commit3c4797ba74e0b7db0db3e34b606ba9c1d0f2baf7 (patch)
treec95a2cf50e696712b6fd29459e01599bb9dd6d05 /gdb
parent71193121ff7a6ee95fff5eb9d8ae378390ecdf45 (diff)
downloadgdb-3c4797ba74e0b7db0db3e34b606ba9c1d0f2baf7.zip
gdb-3c4797ba74e0b7db0db3e34b606ba9c1d0f2baf7.tar.gz
gdb-3c4797ba74e0b7db0db3e34b606ba9c1d0f2baf7.tar.bz2
breakpoint.c:watchpoints_triggered: simplify a tiny bit.
I was reading this, checking the the possible returns, and this particular path confused a tiny little. Above we do: if (!stopped_by_watchpoint) { ... return 0; } so any return after that always return true. Tested on x86_64 Fedora 17. gdb/ 2013-10-28 Pedro Alves <palves@redhat.com> * breakpoint.c (watchpoints_triggered) <!target_stopped_data_address>: Hardcode return 1.
Diffstat (limited to 'gdb')
-rw-r--r--gdb/ChangeLog5
-rw-r--r--gdb/breakpoint.c2
2 files changed, 6 insertions, 1 deletions
diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index c8c2860..8bb0385 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,5 +1,10 @@
2013-10-28 Pedro Alves <palves@redhat.com>
+ * breakpoint.c (watchpoints_triggered)
+ <!target_stopped_data_address>: Hardcode return 1.
+
+2013-10-28 Pedro Alves <palves@redhat.com>
+
* infrun.c (process_event_stop_test): Remove unnecessary scoping
level and reindent.
diff --git a/gdb/breakpoint.c b/gdb/breakpoint.c
index c630b87..e172be1 100644
--- a/gdb/breakpoint.c
+++ b/gdb/breakpoint.c
@@ -4714,7 +4714,7 @@ watchpoints_triggered (struct target_waitstatus *ws)
w->watchpoint_triggered = watch_triggered_unknown;
}
- return stopped_by_watchpoint;
+ return 1;
}
/* The target could report the data address. Mark watchpoints