aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--gdb/ChangeLog7
-rw-r--r--gdb/breakpoint.c6
-rw-r--r--gdb/testsuite/ChangeLog5
-rw-r--r--gdb/testsuite/gdb.mi/mi-break.exp10
4 files changed, 25 insertions, 3 deletions
diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 4cfe1fb..7ddc112 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,10 @@
+2008-02-01 Vladimir Prus <vladimir@codesourcery.com>
+
+ Properly rethrow exception. This fixes errors
+ about non-existent functions for -break-insert.
+ * breakpoint.c (break_command_really): Use throw_exception
+ for rethrowing. If rethrowing, don't print the exception.
+
2008-01-31 Thiago Jung Bauermann <bauerman@br.ibm.com>
* NEWS: Mention Decimal Floating Point support.
diff --git a/gdb/breakpoint.c b/gdb/breakpoint.c
index 0bed4ef..f69002c 100644
--- a/gdb/breakpoint.c
+++ b/gdb/breakpoint.c
@@ -5330,13 +5330,13 @@ break_command_really (char *arg, char *cond_string, int thread,
{
case NOT_FOUND_ERROR:
- exception_print (gdb_stderr, e);
-
/* If pending breakpoint support is turned off, throw
error. */
if (pending_break_support == AUTO_BOOLEAN_FALSE)
- deprecated_throw_reason (RETURN_ERROR);
+ throw_exception (e);
+
+ exception_print (gdb_stderr, e);
/* If pending breakpoint support is auto query and the user
selects no, then simply return the error code. */
diff --git a/gdb/testsuite/ChangeLog b/gdb/testsuite/ChangeLog
index e39f566..99a21e3 100644
--- a/gdb/testsuite/ChangeLog
+++ b/gdb/testsuite/ChangeLog
@@ -1,3 +1,8 @@
+2008-02-01 Vladimir Prus <vladimir@codesourcery.com>
+
+ * gdb.mi/mi-break.exp (test_error): New.
+ Call it.
+
2008-01-31 Luis Machado <luisgpm@br.ibm.com>
* gdb.arch/ppc64-atomic-inst.c: New testcase source file.
diff --git a/gdb/testsuite/gdb.mi/mi-break.exp b/gdb/testsuite/gdb.mi/mi-break.exp
index af438b9..6933a34 100644
--- a/gdb/testsuite/gdb.mi/mi-break.exp
+++ b/gdb/testsuite/gdb.mi/mi-break.exp
@@ -175,10 +175,20 @@ proc test_ignore_count {} {
}
}
+proc test_error {} {
+ global mi_gdb_prompt
+
+ mi_gdb_test "-break-insert function_that_does_not_exist" \
+ ".*\\^error,msg=\"Function \\\\\"function_that_does_not_exist\\\\\" not defined.\"" \
+ "breakpoint at nonexistent function"
+}
+
test_tbreak_creation_and_listing
test_rbreak_creation_and_listing
test_ignore_count
+test_error
+
mi_gdb_exit
return 0