Eliminate exceptions.c:print_any_exception.
authorPedro Alves <palves@redhat.com>
Mon, 14 Jul 2014 18:55:30 +0000 (19:55 +0100)
committerPedro Alves <palves@redhat.com>
Mon, 14 Jul 2014 19:29:30 +0000 (20:29 +0100)
exception_print and exception_fprintf call print_flush, which does all the
same flushing and annotation things that print_any_exception does, and more.

gdb/
2014-07-14  Pedro Alves  <palves@redhat.com>

* exceptions.c (print_any_exception): Delete.
(catch_exceptions_with_msg): Use exception_print instead of
print_any_exception.
(catch_errors): Use exception_fprintf instead of
print_any_exception.
(catch_command_errors, catch_command_errors_const): Use
exception_print instead of print_any_exception.

gdb/ChangeLog
gdb/exceptions.c

index 3d2a4dc..7d686b8 100644 (file)
@@ -1,5 +1,15 @@
 2014-07-14  Pedro Alves  <palves@redhat.com>
 
+       * exceptions.c (print_any_exception): Delete.
+       (catch_exceptions_with_msg): Use exception_print instead of
+       print_any_exception.
+       (catch_errors): Use exception_fprintf instead of
+       print_any_exception.
+       (catch_command_errors, catch_command_errors_const): Use
+       exception_print instead of print_any_exception.
+
+2014-07-14  Pedro Alves  <palves@redhat.com>
+
        * infcall.c (run_inferior_call): Set 'sync_execution' while
        running the inferior call.
 
index ca80893..8ee428d 100644 (file)
@@ -331,24 +331,6 @@ exception_fprintf (struct ui_file *file, struct gdb_exception e,
     }
 }
 
-static void
-print_any_exception (struct ui_file *file, const char *prefix,
-                    struct gdb_exception e)
-{
-  if (e.reason < 0 && e.message != NULL)
-    {
-      target_terminal_ours ();
-      wrap_here ("");          /* Force out any buffered output.  */
-      gdb_flush (gdb_stdout);
-      annotate_error_begin ();
-
-      /* Print the prefix.  */
-      if (prefix != NULL && prefix[0] != '\0')
-       fputs_filtered (prefix, file);
-      print_exception (file, e);
-    }
-}
-
 /* A stack of exception messages.
    This is needed to handle nested calls to throw_it: we don't want to
    xfree space for a message before it's used.
@@ -486,7 +468,7 @@ catch_exceptions_with_msg (struct ui_out *func_uiout,
       throw_exception (exception);
     }
 
-  print_any_exception (gdb_stderr, NULL, exception);
+  exception_print (gdb_stderr, exception);
   gdb_assert (val >= 0);
   gdb_assert (exception.reason <= 0);
   if (exception.reason < 0)
@@ -534,7 +516,7 @@ catch_errors (catch_errors_ftype *func, void *func_args, char *errstring,
       throw_exception (exception);
     }
 
-  print_any_exception (gdb_stderr, errstring, exception);
+  exception_fprintf (gdb_stderr, exception, "%s", errstring);
   if (exception.reason != 0)
     return 0;
   return val;
@@ -550,7 +532,7 @@ catch_command_errors (catch_command_errors_ftype *command,
     {
       command (arg, from_tty);
     }
-  print_any_exception (gdb_stderr, NULL, e);
+  exception_print (gdb_stderr, e);
   if (e.reason < 0)
     return 0;
   return 1;
@@ -566,7 +548,7 @@ catch_command_errors_const (catch_command_errors_const_ftype *command,
     {
       command (arg, from_tty);
     }
-  print_any_exception (gdb_stderr, NULL, e);
+  exception_print (gdb_stderr, e);
   if (e.reason < 0)
     return 0;
   return 1;