[gdb/cli] Improve show logging output
[binutils-gdb.git] / gdb / cli / cli-logging.c
index 29b2c872a785b52f567af47ef2f816644592f97b..193a87331db3244c209172fb663e0cbd0360bda8 100644 (file)
@@ -1,7 +1,6 @@
 /* Command-line output logging for GDB, the GNU debugger.
 
-   Copyright (c) 2003, 2004, 2007, 2008, 2009, 2010
-   Free Software Foundation, Inc.
+   Copyright (C) 2003-2022 Free Software Foundation, Inc.
 
    This file is part of GDB.
 
 #include "defs.h"
 #include "gdbcmd.h"
 #include "ui-out.h"
-#include "gdb_assert.h"
+#include "interps.h"
+#include "cli/cli-style.h"
+#include "cli/cli-decode.h"
 
-#include "gdb_string.h"
+static std::string saved_filename;
 
-/* These hold the pushed copies of the gdb output files.
-   If NULL then nothing has yet been pushed.  */
-struct saved_output_files
-{
-  struct ui_file *out;
-  struct ui_file *err;
-  struct ui_file *log;
-  struct ui_file *targ;
-  struct ui_file *targerr;
-};
-static struct saved_output_files saved_output;
-static char *saved_filename;
-
-static char *logging_filename;
+static std::string logging_filename = "gdb.txt";
 static void
 show_logging_filename (struct ui_file *file, int from_tty,
                       struct cmd_list_element *c, const char *value)
 {
-  fprintf_filtered (file, _("The current logfile is \"%s\".\n"),
-                   value);
+  fprintf_filtered (file, _("The current logfile is \"%ps\".\n"),
+                   styled_string (file_name_style.style (), value));
 }
 
-static int logging_overwrite;
+static bool logging_overwrite;
 
 static void
-set_logging_overwrite (char *args, int from_tty, struct cmd_list_element *c)
+maybe_warn_already_logging ()
 {
-  if (saved_filename)
+  if (!saved_filename.empty ())
     warning (_("Currently logging to %s.  Turn the logging off and on to "
-              "make the new setting effective."), saved_filename);
+              "make the new setting effective."), saved_filename.c_str ());
+}
+
+static void
+set_logging_overwrite (const char *args,
+                      int from_tty, struct cmd_list_element *c)
+{
+  maybe_warn_already_logging ();
 }
 
 static void
 show_logging_overwrite (struct ui_file *file, int from_tty,
                        struct cmd_list_element *c, const char *value)
 {
-  fprintf_filtered (file, _("\
-Whether logging overwrites or appends to the log file is %s.\n"),
-                   value);
+  if (logging_overwrite)
+    fprintf_filtered (file, _("on: Logging overwrites the log file.\n"));
+  else
+    fprintf_filtered (file, _("off: Logging appends to the log file.\n"));
 }
 
 /* Value as configured by the user.  */
-static int logging_redirect;
-
-/* The on-disk file in use if logging is currently active together
-   with redirection turned off (and therefore using tee_file_new).
-   For active logging with redirection the on-disk file is directly in
-   GDB_STDOUT and this variable is NULL.  */
-static struct ui_file *logging_no_redirect_file;
+static bool logging_redirect;
+static bool debug_redirect;
 
 static void
-set_logging_redirect (char *args, int from_tty, struct cmd_list_element *c)
+set_logging_redirect (const char *args,
+                     int from_tty, struct cmd_list_element *c)
 {
-  struct cleanup *cleanups = NULL;
-  struct ui_file *output, *new_logging_no_redirect_file;
-
-  if (saved_filename == NULL
-      || (logging_redirect != 0 && logging_no_redirect_file == NULL)
-      || (logging_redirect == 0 && logging_no_redirect_file != NULL))
-    return;
-
-  if (logging_redirect != 0)
-    {
-      gdb_assert (logging_no_redirect_file != NULL);
-
-      /* ui_out_redirect still has not been called for next
-        gdb_stdout.  */
-      cleanups = make_cleanup_ui_file_delete (gdb_stdout);
-
-      output = logging_no_redirect_file;
-      new_logging_no_redirect_file = NULL;
+  maybe_warn_already_logging ();
+}
 
-      if (from_tty)
-       fprintf_unfiltered (saved_output.out, "Redirecting output to %s.\n",
-                           logging_filename);
-    }
+static void
+show_logging_redirect (struct ui_file *file, int from_tty,
+                      struct cmd_list_element *c, const char *value)
+{
+  if (logging_redirect)
+    fprintf_filtered(file, _("on: Output will go only to the log file.\n"));
   else
-    {
-      gdb_assert (logging_no_redirect_file == NULL);
-      output = tee_file_new (saved_output.out, 0, gdb_stdout, 0);
-      if (output == NULL)
-       perror_with_name (_("set logging"));
-      new_logging_no_redirect_file = gdb_stdout;
-
-      if (from_tty)
-       fprintf_unfiltered (saved_output.out, "Copying output to %s.\n",
-                           logging_filename);
-    }
-
-  gdb_stdout = output;
-  gdb_stderr = output;
-  gdb_stdlog = output;
-  gdb_stdtarg = output;
-  gdb_stdtargerr = output;
-  logging_no_redirect_file = new_logging_no_redirect_file;
-
-  /* There is a former output pushed on the ui_out_redirect stack.  We
-     want to replace it by OUTPUT so we must pop the former value
-     first.  We should either do both the pop and push or to do
-     neither of it.  At least do not try to push OUTPUT if the pop
-     already failed.  */
-
-  if (ui_out_redirect (uiout, NULL) < 0
-      || ui_out_redirect (uiout, output) < 0)
-    warning (_("Current output protocol does not support redirection"));
-
-  if (logging_redirect != 0)
-    do_cleanups (cleanups);
+    fprintf_filtered
+      (file,
+       _("off: Output will go to both the screen and the log file.\n"));
 }
 
 static void
-show_logging_redirect (struct ui_file *file, int from_tty,
+show_logging_debug_redirect (struct ui_file *file, int from_tty,
                       struct cmd_list_element *c, const char *value)
 {
-  fprintf_filtered (file, _("The logging output mode is %s.\n"), value);
+  if (debug_redirect)
+    fprintf_filtered(file,
+                    _("on: Debug output will go only to the log file.\n"));
+  else
+    fprintf_filtered
+      (file,
+       _("off: Debug output will go to both the screen and the log file.\n"));
 }
 
 /* If we've pushed output files, close them and pop them.  */
 static void
 pop_output_files (void)
 {
-  /* Only delete one of the files -- they are all set to the same
-     value.  */
-  ui_file_delete (gdb_stdout);
-  if (logging_no_redirect_file)
-    {
-      ui_file_delete (logging_no_redirect_file);
-      logging_no_redirect_file = NULL;
-    }
-  gdb_stdout = saved_output.out;
-  gdb_stderr = saved_output.err;
-  gdb_stdlog = saved_output.log;
-  gdb_stdtarg = saved_output.targ;
-  gdb_stdtargerr = saved_output.targ;
-  saved_output.out = NULL;
-  saved_output.err = NULL;
-  saved_output.log = NULL;
-  saved_output.targ = NULL;
-  saved_output.targerr = NULL;
-
-  ui_out_redirect (uiout, NULL);
+  current_interp_set_logging (NULL, false, false);
+
+  /* Stay consistent with handle_redirections.  */
+  if (!current_uiout->is_mi_like_p ())
+    current_uiout->redirect (NULL);
 }
 
 /* This is a helper for the `set logging' command.  */
 static void
 handle_redirections (int from_tty)
 {
-  struct cleanup *cleanups;
-  struct ui_file *output;
-
-  if (saved_filename != NULL)
+  if (!saved_filename.empty ())
     {
       fprintf_unfiltered (gdb_stdout, "Already logging to %s.\n",
-                         saved_filename);
+                         saved_filename.c_str ());
       return;
     }
 
-  output = gdb_fopen (logging_filename, logging_overwrite ? "w" : "a");
-  if (output == NULL)
+  stdio_file_up log (new no_terminal_escape_file ());
+  if (!log->open (logging_filename.c_str (), logging_overwrite ? "w" : "a"))
     perror_with_name (_("set logging"));
-  cleanups = make_cleanup_ui_file_delete (output);
 
   /* Redirects everything to gdb_stdout while this is running.  */
-  if (!logging_redirect)
+  if (from_tty)
     {
-      struct ui_file *no_redirect_file = output;
-
-      output = tee_file_new (gdb_stdout, 0, no_redirect_file, 0);
-      if (output == NULL)
-       perror_with_name (_("set logging"));
-      make_cleanup_ui_file_delete (output);
-      if (from_tty)
+      if (!logging_redirect)
        fprintf_unfiltered (gdb_stdout, "Copying output to %s.\n",
-                           logging_filename);
-      logging_no_redirect_file = no_redirect_file;
-    }
-  else
-    {
-      gdb_assert (logging_no_redirect_file == NULL);
-
-      if (from_tty)
+                           logging_filename.c_str ());
+      else
        fprintf_unfiltered (gdb_stdout, "Redirecting output to %s.\n",
-                           logging_filename);
-    }
+                           logging_filename.c_str ());
 
-  discard_cleanups (cleanups);
+      if (!debug_redirect)
+       fprintf_unfiltered (gdb_stdout, "Copying debug output to %s.\n",
+                           logging_filename.c_str ());
+      else
+       fprintf_unfiltered (gdb_stdout, "Redirecting debug output to %s.\n",
+                           logging_filename.c_str ());
+    }
 
-  saved_filename = xstrdup (logging_filename);
-  saved_output.out = gdb_stdout;
-  saved_output.err = gdb_stderr;
-  saved_output.log = gdb_stdlog;
-  saved_output.targ = gdb_stdtarg;
-  saved_output.targerr = gdb_stdtargerr;
+  saved_filename = logging_filename;
 
-  gdb_stdout = output;
-  gdb_stderr = output;
-  gdb_stdlog = output;
-  gdb_stdtarg = output;
-  gdb_stdtargerr = output;
+  /* Let the interpreter do anything it needs.  */
+  current_interp_set_logging (std::move (log), logging_redirect,
+                             debug_redirect);
 
-  if (ui_out_redirect (uiout, output) < 0)
-    warning (_("Current output protocol does not support redirection"));
+  /* Redirect the current ui-out object's output to the log.  Use
+     gdb_stdout, not log, since the interpreter may have created a tee
+     that wraps the log.  Don't do the redirect for MI, it confuses
+     MI's ui-out scheme.  Note that we may get here with MI as current
+     interpreter, but with the current ui_out as a CLI ui_out, with
+     '-interpreter-exec console "set logging on"'.  */
+  if (!current_uiout->is_mi_like_p ())
+    current_uiout->redirect (gdb_stdout);
 }
 
 static void
-set_logging_on (char *args, int from_tty)
+set_logging_on (const char *args, int from_tty)
 {
-  char *rest = args;
+  const char *rest = args;
 
   if (rest && *rest)
-    {
-      xfree (logging_filename);
-      logging_filename = xstrdup (rest);
-    }
+    logging_filename = rest;
+
   handle_redirections (from_tty);
 }
 
 static void 
-set_logging_off (char *args, int from_tty)
+set_logging_off (const char *args, int from_tty)
 {
-  if (saved_filename == NULL)
+  if (saved_filename.empty ())
     return;
 
   pop_output_files ();
   if (from_tty)
-    fprintf_unfiltered (gdb_stdout, "Done logging to %s.\n", saved_filename);
-  xfree (saved_filename);
-  saved_filename = NULL;
+    fprintf_unfiltered (gdb_stdout, "Done logging to %s.\n",
+                       saved_filename.c_str ());
+  saved_filename.clear ();
 }
 
+static bool logging_enabled;
+
 static void
-set_logging_command (char *args, int from_tty)
+set_logging_enabled (const char *args,
+                    int from_tty, struct cmd_list_element *c)
 {
-  printf_unfiltered (_("\
-\"set logging\" lets you log output to a file.\n\
-Usage: set logging on [FILENAME]\n\
-       set logging off\n\
-       set logging file FILENAME\n\
-       set logging overwrite [on|off]\n\
-       set logging redirect [on|off]\n"));
+  if (logging_enabled)
+    set_logging_on (args, from_tty);
+  else
+    set_logging_off (args, from_tty);
 }
 
 static void
-show_logging_command (char *args, int from_tty)
+show_logging_enabled (struct ui_file *file, int from_tty,
+                      struct cmd_list_element *c, const char *value)
 {
-  if (saved_filename)
-    printf_unfiltered (_("Currently logging to \"%s\".\n"), saved_filename);
-  if (saved_filename == NULL
-      || strcmp (logging_filename, saved_filename) != 0)
-    printf_unfiltered (_("Future logs will be written to %s.\n"),
-                      logging_filename);
-
-  if (logging_overwrite)
-    printf_unfiltered (_("Logs will overwrite the log file.\n"));
+  if (logging_enabled)
+    fprintf_unfiltered (_("on: Logging is enabled.\n"));
   else
-    printf_unfiltered (_("Logs will be appended to the log file.\n"));
-
-  if (saved_filename)
-    {
-      if (logging_redirect)
-       printf_unfiltered (_("Output is being sent only to the log file.\n"));
-      else
-       printf_unfiltered (_("Output is being logged and displayed.\n"));
-    }
-  else
-    {
-      if (logging_redirect)
-       printf_unfiltered (_("Output will be sent only to the log file.\n"));
-      else
-       printf_unfiltered (_("Output will be logged and displayed.\n"));
-    }
+    fprintf_unfiltered (_("off: Logging is disabled.\n"));
 }
 
-/* Provide a prototype to silence -Wmissing-prototypes.  */
-extern initialize_file_ftype _initialize_cli_logging;
-
+void _initialize_cli_logging ();
 void
-_initialize_cli_logging (void)
+_initialize_cli_logging ()
 {
   static struct cmd_list_element *set_logging_cmdlist, *show_logging_cmdlist;
 
-  add_prefix_cmd ("logging", class_support, set_logging_command,
-                 _("Set logging options"), &set_logging_cmdlist,
-                 "set logging ", 0, &setlist);
-  add_prefix_cmd ("logging", class_support, show_logging_command,
-                 _("Show logging options"), &show_logging_cmdlist,
-                 "show logging ", 0, &showlist);
+  /* Set/show logging.  */
+  add_setshow_prefix_cmd ("logging", class_support,
+                         _("Set logging options."),
+                         _("Show logging options."),
+                         &set_logging_cmdlist, &show_logging_cmdlist,
+                         &setlist, &showlist);
+
+  /* Set/show logging overwrite.  */
   add_setshow_boolean_cmd ("overwrite", class_support, &logging_overwrite, _("\
 Set whether logging overwrites or appends to the log file."), _("\
 Show whether logging overwrites or appends to the log file."), _("\
-If set, logging overrides the log file."),
+If set, logging overwrites the log file."),
                           set_logging_overwrite,
                           show_logging_overwrite,
                           &set_logging_cmdlist, &show_logging_cmdlist);
+
+  /* Set/show logging redirect.  */
   add_setshow_boolean_cmd ("redirect", class_support, &logging_redirect, _("\
 Set the logging output mode."), _("\
 Show the logging output mode."), _("\
@@ -327,6 +235,19 @@ If redirect is on, output will go only to the log file."),
                           set_logging_redirect,
                           show_logging_redirect,
                           &set_logging_cmdlist, &show_logging_cmdlist);
+
+  /* Set/show logging debugredirect.  */
+  add_setshow_boolean_cmd ("debugredirect", class_support,
+                          &debug_redirect, _("\
+Set the logging debug output mode."), _("\
+Show the logging debug output mode."), _("\
+If debug redirect is off, debug will go to both the screen and the log file.\n\
+If debug redirect is on, debug will go only to the log file."),
+                          set_logging_redirect,
+                          show_logging_debug_redirect,
+                          &set_logging_cmdlist, &show_logging_cmdlist);
+
+  /* Set/show logging file.  */
   add_setshow_filename_cmd ("file", class_support, &logging_filename, _("\
 Set the current logfile."), _("\
 Show the current logfile."), _("\
@@ -334,10 +255,28 @@ The logfile is used when directing GDB's output."),
                            NULL,
                            show_logging_filename,
                            &set_logging_cmdlist, &show_logging_cmdlist);
-  add_cmd ("on", class_support, set_logging_on,
-          _("Enable logging."), &set_logging_cmdlist);
-  add_cmd ("off", class_support, set_logging_off,
-          _("Disable logging."), &set_logging_cmdlist);
 
-  logging_filename = xstrdup ("gdb.txt");
+  /* Set/show logging enabled.  */
+  set_show_commands setshow_logging_enabled_cmds
+    = add_setshow_boolean_cmd ("enabled", class_support, &logging_enabled,
+                              _("Enable logging."),
+                              _("Show whether logging is enabled."),
+                              _("When on, enable logging."),
+                              set_logging_enabled,
+                              show_logging_enabled,
+                              &set_logging_cmdlist, &show_logging_cmdlist);
+
+  /* Set logging on, deprecated alias.  */
+  cmd_list_element *set_logging_on_cmd
+    = add_alias_cmd ("on", setshow_logging_enabled_cmds.set, class_support,
+                    false, &set_logging_cmdlist);
+  deprecate_cmd (set_logging_on_cmd, "set logging enabled on");
+  set_logging_on_cmd->default_args = "on";
+
+  /* Set logging off, deprecated alias.  */
+  cmd_list_element *set_logging_off_cmd
+    = add_alias_cmd ("off", setshow_logging_enabled_cmds.set, class_support,
+                    false, &set_logging_cmdlist);
+  deprecate_cmd (set_logging_off_cmd, "set logging enabled off");
+  set_logging_off_cmd->default_args = "off";
 }