Fix MI timings.

        * mi/mi-main.c (mi_print_timing_maybe): New.
        (captured_mi_execute_command): Simplify. Output timings to
        CLI commands, too.
        (mi_execute_async_cli_command): Do not print timings.
        * mi/mi-main.h (mi_print_timing_maybe): Declare.
        * mi/mi-interp.c (mi_on_normal_stop): Call mi_print_timing_maybe.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index fc0f7d285861455d44966e44db01d8a40df792a1..9e6ecda692d829663080ee294f492c3fc8ff461e 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,14 @@
+2009-03-12  Vladimir Prus  <vladimir@codesourcery.com>
+
+       Fix MI timings.
+
+        * mi/mi-main.c (mi_print_timing_maybe): New.
+        (captured_mi_execute_command): Simplify. Output timings to
+        CLI commands, too.
+        (mi_execute_async_cli_command): Do not print timings.
+        * mi/mi-main.h (mi_print_timing_maybe): Declare.
+        * mi/mi-interp.c (mi_on_normal_stop): Call mi_print_timing_maybe.
+
 2009-03-12  Jerome Guitton  <guitton@adacore.com>
 
        * xcoffread.c (process_linenos): Check if the line in the

diff --git a/gdb/mi/mi-interp.c b/gdb/mi/mi-interp.c
index 65c1b4f49fafb5da3ba1ff05616a9026b4e3982e..131698b34893b5bac72181f101a240760cf2bb08 100644 (file)
--- a/gdb/mi/mi-interp.c
+++ b/gdb/mi/mi-interp.c
@@ -362,6 +362,7 @@ mi_on_normal_stop (struct bpstats *bs, int print_frame)
   fputs_unfiltered ("*stopped", raw_stdout);
   mi_out_put (mi_uiout, raw_stdout);
   mi_out_rewind (mi_uiout);
+  mi_print_timing_maybe ();
   fputs_unfiltered ("\n", raw_stdout);
   gdb_flush (raw_stdout);
 }

diff --git a/gdb/mi/mi-main.c b/gdb/mi/mi-main.c
index b905a9e48ae4dd97c8c532d66de3919b12a50e50..2873b4d6d8e67daa65683d9970546ab036d9d0c6 100644 (file)
--- a/gdb/mi/mi-main.c
+++ b/gdb/mi/mi-main.c
@@ -1139,7 +1139,8 @@ captured_mi_execute_command (struct ui_out *uiout, void *data)
 {
   struct mi_parse *context = (struct mi_parse *) data;
 
-  struct mi_timestamp cmd_finished;
+  if (do_timings)
+    current_command_ts = context->cmd_start;
 
   running_result_record_printed = 0;
   switch (context->op)
@@ -1151,14 +1152,9 @@ captured_mi_execute_command (struct ui_out *uiout, void *data)
        fprintf_unfiltered (raw_stdout, " token=`%s' command=`%s' args=`%s'\n",
                            context->token, context->command, context->args);
 
-      if (do_timings)
-       current_command_ts = context->cmd_start;
 
       mi_cmd_execute (context);
 
-      if (do_timings)
-       timestamp (&cmd_finished);
-
       /* Print the result if there were no errors.
 
         Remember that on the way out of executing a command, you have
@@ -1174,10 +1170,7 @@ captured_mi_execute_command (struct ui_out *uiout, void *data)
                            ? "^connected" : "^done", raw_stdout);
          mi_out_put (uiout, raw_stdout);
          mi_out_rewind (uiout);
-         /* Have to check cmd_start, since the command could be
-            -enable-timings.  */
-         if (do_timings && context->cmd_start)
-           print_diff (context->cmd_start, &cmd_finished);
+         mi_print_timing_maybe ();
          fputs_unfiltered ("\n", raw_stdout);
        }
       else
@@ -1212,7 +1205,8 @@ captured_mi_execute_command (struct ui_out *uiout, void *data)
                fputs_unfiltered ("^done", raw_stdout);
                mi_out_put (uiout, raw_stdout);
                mi_out_rewind (uiout);
-               fputs_unfiltered ("\n", raw_stdout);
+               mi_print_timing_maybe ();
+               fputs_unfiltered ("\n", raw_stdout);            
              }
            else
              mi_out_rewind (uiout);
@@ -1447,8 +1441,6 @@ mi_execute_async_cli_command (char *cli_command, char **argv, int argc)
       /* Do this before doing any printing.  It would appear that some
          print code leaves garbage around in the buffer.  */
       do_cleanups (old_cleanups);
-      if (do_timings)
-       print_diff_now (current_command_ts);
     }
 }
 
@@ -1567,6 +1559,15 @@ print_diff_now (struct mi_timestamp *start)
     print_diff (start, &now);
   }
 
+void
+mi_print_timing_maybe (void)
+{
+  /* If the command is -enable-timing then do_timings may be
+     true whilst current_command_ts is not initialized.  */
+  if (do_timings && current_command_ts)
+    print_diff_now (current_command_ts);
+}
+
 static long 
 timeval_diff (struct timeval start, struct timeval end)
   {

diff --git a/gdb/mi/mi-main.h b/gdb/mi/mi-main.h
index 977579e25295b5b69501a81f225a3a88686deeca..1383177f27aa2de69d175e6fb4ffca581761ea53 100644 (file)
--- a/gdb/mi/mi-main.h
+++ b/gdb/mi/mi-main.h
@@ -30,5 +30,7 @@ extern char *current_token;
 
 extern int running_result_record_printed;
 
+void mi_print_timing_maybe ();
+
 #endif