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.

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.

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.  */

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.

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