]> git.ipfire.org Git - thirdparty/binutils-gdb.git/blobdiff - gdb/command.c
import gdb-1999-07-07 post reformat
[thirdparty/binutils-gdb.git] / gdb / command.c
index 0b095d9afad59e2ffdbcd3bf937f36d064646ba7..ad3e7e490b286ef49c480e27ed131791eb4970e5 100644 (file)
@@ -1,19 +1,20 @@
 /* Handle lists of commands, their decoding and documentation, for GDB.
    Copyright 1986, 1989, 1990, 1991, 1998 Free Software Foundation, Inc.
 
-This program is free software; you can redistribute it and/or modify
-it under the terms of the GNU General Public License as published by
-the Free Software Foundation; either version 2 of the License, or
-(at your option) any later version.
+   This program is free software; you can redistribute it and/or modify
+   it under the terms of the GNU General Public License as published by
+   the Free Software Foundation; either version 2 of the License, or
+   (at your option) any later version.
 
-This program is distributed in the hope that it will be useful,
-but WITHOUT ANY WARRANTY; without even the implied warranty of
-MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-GNU General Public License for more details.
+   This program is distributed in the hope that it will be useful,
+   but WITHOUT ANY WARRANTY; without even the implied warranty of
+   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+   GNU General Public License for more details.
 
-You should have received a copy of the GNU General Public License
-along with this program; if not, write to the Free Software
-Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.  */
+   You should have received a copy of the GNU General Public License
+   along with this program; if not, write to the Free Software
+   Foundation, Inc., 59 Temple Place - Suite 330,
+   Boston, MA 02111-1307, USA.  */
 
 #include "defs.h"
 #include "gdbcmd.h"
@@ -26,11 +27,11 @@ Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.  */
 #endif
 
 #ifdef HAVE_WAIT_H
-# include <wait.h>
+#include <wait.h>
 #else
-# ifdef HAVE_SYS_WAIT_H
-#  include <sys/wait.h>
-# endif
+#ifdef HAVE_SYS_WAIT_H
+#include <sys/wait.h>
+#endif
 #endif
 
 #include "wait.h"
@@ -53,7 +54,7 @@ static void print_doc_line PARAMS ((GDB_FILE *, char *));
 
 static struct cmd_list_element *find_cmd PARAMS ((char *command,
                                                  int len,
-                                                 struct cmd_list_element *clist,
+                                           struct cmd_list_element * clist,
                                                  int ignore_help_classes,
                                                  int *nfound));
 
@@ -85,7 +86,7 @@ add_cmd (name, class, fun, doc, list)
      struct cmd_list_element **list;
 {
   register struct cmd_list_element *c
-    = (struct cmd_list_element *) xmalloc (sizeof (struct cmd_list_element));
+  = (struct cmd_list_element *) xmalloc (sizeof (struct cmd_list_element));
   struct cmd_list_element *p;
 
   delete_cmd (name, list);
@@ -99,9 +100,9 @@ add_cmd (name, class, fun, doc, list)
     {
       p = *list;
       while (p->next && STRCMP (p->next->name, name) <= 0)
-        {
-          p = p->next;
-        }
+       {
+         p = p->next;
+       }
       c->next = p->next;
       p->next = c;
     }
@@ -129,7 +130,7 @@ add_cmd (name, class, fun, doc, list)
 
 /* Same as above, except that the abbrev_flag is set. */
 
-#if 0  /* Currently unused */
+#if 0                          /* Currently unused */
 
 struct cmd_list_element *
 add_abbrev_cmd (name, class, fun, doc, list)
@@ -140,7 +141,7 @@ add_abbrev_cmd (name, class, fun, doc, list)
      struct cmd_list_element **list;
 {
   register struct cmd_list_element *c
-    = add_cmd (name, class, fun, doc, list);
+  = add_cmd (name, class, fun, doc, list);
 
   c->abbrev_flag = 1;
   return c;
@@ -162,7 +163,7 @@ add_alias_cmd (name, oldname, class, abbrev_flag, list)
   register struct cmd_list_element *c;
   copied_name = (char *) alloca (strlen (oldname) + 1);
   strcpy (copied_name, oldname);
-  old  = lookup_cmd (&copied_name, *list, "", 1, 1);
+  old = lookup_cmd (&copied_name, *list, "", 1, 1);
 
   if (old == 0)
     {
@@ -204,7 +205,7 @@ add_prefix_cmd (name, class, fun, doc, prefixlist, prefixname,
 }
 
 /* Like add_prefix_cmd but sets the abbrev_flag on the new command. */
-   
+
 struct cmd_list_element *
 add_abbrev_prefix_cmd (name, class, fun, doc, prefixlist, prefixname,
                       allow_unknown, list)
@@ -261,7 +262,7 @@ add_set_cmd (name, class, var_type, var, doc, list)
      struct cmd_list_element **list;
 {
   struct cmd_list_element *c
-    = add_cmd (name, class, NO_FUNCTION, doc, list);
+  = add_cmd (name, class, NO_FUNCTION, doc, list);
 
   c->type = set_cmd;
   c->var_type = var_type;
@@ -277,7 +278,7 @@ add_set_cmd (name, class, var_type, var, doc, list)
    CLASS is as in add_cmd.
    ENUMLIST is a list of strings which may follow NAME.
    VAR is address of the variable which will contain the matching string
-     (from ENUMLIST).
+   (from ENUMLIST).
    DOC is the documentation string.  */
 
 struct cmd_list_element *
@@ -290,7 +291,7 @@ add_set_enum_cmd (name, class, enumlist, var, doc, list)
      struct cmd_list_element **list;
 {
   struct cmd_list_element *c
-    = add_set_cmd (name, class, var_enum, var, doc, list);
+  = add_set_cmd (name, class, var_enum, var, doc, list);
   c->enums = enumlist;
 
   return c;
@@ -305,21 +306,21 @@ add_show_from_set (setcmd, list)
      struct cmd_list_element **list;
 {
   struct cmd_list_element *showcmd =
-    (struct cmd_list_element *) xmalloc (sizeof (struct cmd_list_element));
+  (struct cmd_list_element *) xmalloc (sizeof (struct cmd_list_element));
   struct cmd_list_element *p;
 
   memcpy (showcmd, setcmd, sizeof (struct cmd_list_element));
   delete_cmd (showcmd->name, list);
   showcmd->type = show_cmd;
-  
+
   /* Replace "set " at start of docstring with "show ".  */
   if (setcmd->doc[0] == 'S' && setcmd->doc[1] == 'e'
       && setcmd->doc[2] == 't' && setcmd->doc[3] == ' ')
     showcmd->doc = concat ("Show ", setcmd->doc + 4, NULL);
   else
     fprintf_unfiltered (gdb_stderr, "GDB internal error: Bad docstring for set command\n");
-  
-    if (*list == NULL || STRCMP ((*list)->name, showcmd->name) >= 0)
+
+  if (*list == NULL || STRCMP ((*list)->name, showcmd->name) >= 0)
     {
       showcmd->next = *list;
       *list = showcmd;
@@ -328,9 +329,9 @@ add_show_from_set (setcmd, list)
     {
       p = *list;
       while (p->next && STRCMP (p->next->name, showcmd->name) <= 0)
-        {
-          p = p->next;
-        }
+       {
+         p = p->next;
+       }
       showcmd->next = p->next;
       p->next = showcmd;
     }
@@ -353,7 +354,7 @@ delete_cmd (name, list)
       if ((*list)->hookee)
        (*list)->hookee->hook = 0;      /* Hook slips out of its mouth */
       p = (*list)->next;
-      free ((PTR)*list);
+      free ((PTR) * list);
       *list = p;
     }
 
@@ -363,9 +364,9 @@ delete_cmd (name, list)
        if (STREQ (c->next->name, name))
          {
            if (c->next->hookee)
-             c->next->hookee->hook = 0;  /* hooked cmd gets away.  */
+             c->next->hookee->hook = 0;        /* hooked cmd gets away.  */
            p = c->next->next;
-           free ((PTR)c->next);
+           free ((PTR) c->next);
            c->next = p;
          }
        else
@@ -406,10 +407,10 @@ help_cmd (command, stream)
   /* There are three cases here.
      If c->prefixlist is nonzero, we have a prefix command.
      Print its documentation, then list its subcommands.
-     
+
      If c->function is nonzero, we really have a command.
      Print its documentation and return.
-     
+
      If c->function is zero, we have a class name.
      Print its documentation (as if it were a command)
      and then set class to the number of this class
@@ -456,7 +457,7 @@ help_list (list, cmdtype, class, stream)
 {
   int len;
   char *cmdtype1, *cmdtype2;
-  
+
   /* If CMDTYPE is "foo ", CMDTYPE1 gets " foo" and CMDTYPE2 gets "foo sub"  */
   len = strlen (cmdtype);
   cmdtype1 = (char *) alloca (len + 1);
@@ -477,19 +478,19 @@ help_list (list, cmdtype, class, stream)
   else
     fprintf_filtered (stream, "List of %scommands:\n\n", cmdtype2);
 
-  help_cmd_list (list, class, cmdtype, (int)class >= 0, stream);
+  help_cmd_list (list, class, cmdtype, (int) class >= 0, stream);
 
   if (class == all_classes)
     fprintf_filtered (stream, "\n\
 Type \"help%s\" followed by a class name for a list of commands in that class.",
-            cmdtype1);
+                     cmdtype1);
 
   fprintf_filtered (stream, "\n\
 Type \"help%s\" followed by %scommand name for full documentation.\n\
 Command name abbreviations are allowed if unambiguous.\n",
-          cmdtype1, cmdtype2);
+                   cmdtype1, cmdtype2);
 }
-     
+
 /* Print only the first line of STR on STREAM.  */
 static void
 print_doc_line (stream, str)
@@ -512,7 +513,7 @@ print_doc_line (stream, str)
   if (p - str > line_size - 1)
     {
       line_size = p - str + 1;
-      free ((PTR)line_buffer);
+      free ((PTR) line_buffer);
       line_buffer = (char *) xmalloc (line_size);
     }
   strncpy (line_buffer, str, p - str);
@@ -529,10 +530,10 @@ print_doc_line (stream, str)
  * PREFIX is the prefix to print before each command name.
  * STREAM is the stream upon which the output should be written.
  * CLASS should be:
- *     A non-negative class number to list only commands in that
+ *      A non-negative class number to list only commands in that
  * class.
- *     ALL_COMMANDS to list all commands in list.
- *     ALL_CLASSES  to list all classes in list.
+ *      ALL_COMMANDS to list all commands in list.
+ *      ALL_CLASSES  to list all classes in list.
  *
  *   Note that RECURSE will be active on *all* sublists, not just the
  * ones selected by the criteria above (ie. the selection mechanism
@@ -552,8 +553,8 @@ help_cmd_list (list, class, prefix, recurse, stream)
     {
       if (c->abbrev_flag == 0 &&
          (class == all_commands
-         || (class == all_classes && c->function.cfunc == NULL)
-         || (class == c->class && c->function.cfunc != NULL)))
+          || (class == all_classes && c->function.cfunc == NULL)
+          || (class == c->class && c->function.cfunc != NULL)))
        {
          fprintf_filtered (stream, "%s%s -- ", prefix, c->name);
          print_doc_line (stream, c->doc);
@@ -565,14 +566,14 @@ help_cmd_list (list, class, prefix, recurse, stream)
        help_cmd_list (*c->prefixlist, class, c->prefixname, 1, stream);
     }
 }
-
 \f
+
 /* Search the input clist for 'command'.  Return the command if
    found (or NULL if not), and return the number of commands
    found in nfound */
 
 static struct cmd_list_element *
-find_cmd(command, len, clist, ignore_help_classes, nfound)
+find_cmd (command, len, clist, ignore_help_classes, nfound)
      char *command;
      int len;
      struct cmd_list_element *clist;
@@ -581,19 +582,19 @@ find_cmd(command, len, clist, ignore_help_classes, nfound)
 {
   struct cmd_list_element *found, *c;
 
-  found = (struct cmd_list_element *)NULL;
+  found = (struct cmd_list_element *) NULL;
   *nfound = 0;
   for (c = clist; c; c = c->next)
     if (!strncmp (command, c->name, len)
-        && (!ignore_help_classes || c->function.cfunc))
+       && (!ignore_help_classes || c->function.cfunc))
       {
-        found = c;
-        (*nfound)++;
-        if (c->name[len] == '\0')
-          {
-            *nfound = 1;
-            break;
-          }
+       found = c;
+       (*nfound)++;
+       if (c->name[len] == '\0')
+         {
+           *nfound = 1;
+           break;
+         }
       }
   return found;
 }
@@ -629,7 +630,7 @@ find_cmd(command, len, clist, ignore_help_classes, nfound)
    affect the operation).
 
    This routine does *not* modify the text pointed to by TEXT.
-   
+
    If IGNORE_HELP_CLASSES is nonzero, ignore any command list elements which
    are actually help classes rather than commands (i.e. the function field of
    the struct cmd_list_element is NULL).  */
@@ -651,7 +652,7 @@ lookup_cmd_1 (text, clist, result_list, ignore_help_classes)
      so that "set args_foo()" doesn't get interpreted as
      "set args _foo()".  */
   for (p = *text;
-       *p && (isalnum(*p) || *p == '-' || *p == '_' ||
+       *p && (isalnum (*p) || *p == '-' || *p == '_' ||
              (tui_version &&
               (*p == '+' || *p == '<' || *p == '>' || *p == '$')) ||
              (xdb_commands && (*p == '!' || *p == '/' || *p == '?')));
@@ -661,7 +662,7 @@ lookup_cmd_1 (text, clist, result_list, ignore_help_classes)
   /* If nothing but whitespace, return 0.  */
   if (p == *text)
     return 0;
-  
+
   len = p - *text;
 
   /* *text and p now bracket the first command word to lookup (and
@@ -679,20 +680,20 @@ lookup_cmd_1 (text, clist, result_list, ignore_help_classes)
   /* Look it up.  */
   found = 0;
   nfound = 0;
-  found = find_cmd(command, len, clist, ignore_help_classes, &nfound);
+  found = find_cmd (command, len, clist, ignore_help_classes, &nfound);
 
   /* 
-  ** We didn't find the command in the entered case, so lower case it
-  ** and search again.
-  */
+     ** We didn't find the command in the entered case, so lower case it
+     ** and search again.
+   */
   if (!found || nfound == 0)
     {
       for (tmp = 0; tmp < len; tmp++)
-        {
-          char x = command[tmp];
-          command[tmp] = isupper(x) ? tolower(x) : x;
-        }
-      found = find_cmd(command, len, clist, ignore_help_classes, &nfound);
+       {
+         char x = command[tmp];
+         command[tmp] = isupper (x) ? tolower (x) : x;
+       }
+      found = find_cmd (command, len, clist, ignore_help_classes, &nfound);
     }
 
   /* If nothing matches, we have a simple failure.  */
@@ -704,8 +705,8 @@ lookup_cmd_1 (text, clist, result_list, ignore_help_classes)
       if (result_list != NULL)
        /* Will be modified in calling routine
           if we know what the prefix command is.  */
-       *result_list = 0;               
-      return (struct cmd_list_element *) -1; /* Ambiguous.  */
+       *result_list = 0;
+      return (struct cmd_list_element *) -1;   /* Ambiguous.  */
     }
 
   /* We've matched something on this list.  Move text pointer forward. */
@@ -738,9 +739,9 @@ lookup_cmd_1 (text, clist, result_list, ignore_help_classes)
          if (result_list != NULL)
            if (!*result_list)
              /* This used to say *result_list = *found->prefixlist
-                If that was correct, need to modify the documentation
-                at the top of this function to clarify what is supposed
-                to be going on.  */
+                If that was correct, need to modify the documentation
+                at the top of this function to clarify what is supposed
+                to be going on.  */
              *result_list = found;
          return c;
        }
@@ -765,11 +766,11 @@ undef_cmd_error (cmdtype, q)
      char *cmdtype, *q;
 {
   error ("Undefined %scommand: \"%s\".  Try \"help%s%.*s\".",
-    cmdtype,
-    q,
-    *cmdtype? " ": "",
-    strlen(cmdtype)-1,
-    cmdtype);
+        cmdtype,
+        q,
+        *cmdtype ? " " : "",
+        strlen (cmdtype) - 1,
+        cmdtype);
 }
 
 /* Look up the contents of *LINE as a command in the command list LIST.
@@ -796,7 +797,7 @@ lookup_cmd (line, list, cmdtype, allow_unknown, ignore_help_classes)
 {
   struct cmd_list_element *last_list = 0;
   struct cmd_list_element *c =
-    lookup_cmd_1 (line, list, &last_list, ignore_help_classes);
+  lookup_cmd_1 (line, list, &last_list, ignore_help_classes);
 #if 0
   /* This is wrong for complete_command.  */
   char *ptr = (*line) + strlen (*line) - 1;
@@ -806,7 +807,7 @@ lookup_cmd (line, list, cmdtype, allow_unknown, ignore_help_classes)
     ptr--;
   *(ptr + 1) = '\0';
 #endif
-  
+
   if (!c)
     {
       if (!allow_unknown)
@@ -817,7 +818,7 @@ lookup_cmd (line, list, cmdtype, allow_unknown, ignore_help_classes)
            {
              char *p = *line, *q;
 
-             while (isalnum(*p) || *p == '-')
+             while (isalnum (*p) || *p == '-')
                p++;
 
              q = (char *) alloca (p - *line + 1);
@@ -832,13 +833,13 @@ lookup_cmd (line, list, cmdtype, allow_unknown, ignore_help_classes)
   else if (c == (struct cmd_list_element *) -1)
     {
       /* Ambigous.  Local values should be off prefixlist or called
-        values.  */
+         values.  */
       int local_allow_unknown = (last_list ? last_list->allow_unknown :
                                 allow_unknown);
       char *local_cmdtype = last_list ? last_list->prefixname : cmdtype;
       struct cmd_list_element *local_list =
-       (last_list ? *(last_list->prefixlist) : list);
-      
+      (last_list ? *(last_list->prefixlist) : list);
+
       if (local_allow_unknown < 0)
        {
          if (last_list)
@@ -857,12 +858,12 @@ lookup_cmd (line, list, cmdtype, allow_unknown, ignore_help_classes)
                && (*line)[amb_len] != '\t');
               amb_len++)
            ;
-         
+
          ambbuf[0] = 0;
          for (c = local_list; c; c = c->next)
            if (!strncmp (*line, c->name, amb_len))
              {
-               if (strlen (ambbuf) + strlen (c->name) + 6 < (int)sizeof ambbuf)
+               if (strlen (ambbuf) + strlen (c->name) + 6 < (int) sizeof ambbuf)
                  {
                    if (strlen (ambbuf))
                      strcat (ambbuf, ", ");
@@ -894,7 +895,7 @@ lookup_cmd (line, list, cmdtype, allow_unknown, ignore_help_classes)
     }
   return 0;
 }
-       
+
 #if 0
 /* Look up the contents of *LINE as a command in the command list LIST.
    LIST is a chain of struct cmd_list_element's.
@@ -935,7 +936,7 @@ lookup_cmd (line, list, cmdtype, allow_unknown)
   /* Find end of command name.  */
 
   p = *line;
-  while (*p == '-' || isalnum(*p))
+  while (*p == '-' || isalnum (*p))
     p++;
 
   /* Look up the command name.
@@ -950,18 +951,18 @@ lookup_cmd (line, list, cmdtype, allow_unknown)
        error ("Lack of needed %scommand", cmdtype);
       return 0;
     }
-  
+
   /* Copy over to a local buffer, converting to lowercase on the way.
      This is in case the command being parsed is a subcommand which
      doesn't match anything, and that's ok.  We want the original
      untouched for the routine of the original command.  */
-  
+
   processed_cmd = (char *) alloca (p - *line + 1);
   for (cmd_len = 0; cmd_len < p - *line; cmd_len++)
     {
       char x = (*line)[cmd_len];
-      if (isupper(x))
-       processed_cmd[cmd_len] = tolower(x);
+      if (isupper (x))
+       processed_cmd[cmd_len] = tolower (x);
       else
        processed_cmd[cmd_len] = x;
     }
@@ -1016,7 +1017,8 @@ lookup_cmd (line, list, cmdtype, allow_unknown)
 
   /* Skip whitespace before the argument.  */
 
-  while (*p == ' ' || *p == '\t') p++;
+  while (*p == ' ' || *p == '\t')
+    p++;
   *line = p;
 
   if (found->prefixlist && *p)
@@ -1066,12 +1068,12 @@ complete_on_cmdlist (list, text, word)
        if (matches == sizeof_matchlist)
          {
            sizeof_matchlist *= 2;
-           matchlist = (char **) xrealloc ((char *)matchlist,
+           matchlist = (char **) xrealloc ((char *) matchlist,
                                            (sizeof_matchlist
                                             * sizeof (char *)));
          }
 
-       matchlist[matches] = (char *) 
+       matchlist[matches] = (char *)
          xmalloc (strlen (word) + strlen (ptr->name) + 1);
        if (word == text)
          strcpy (matchlist[matches], ptr->name);
@@ -1092,13 +1094,13 @@ complete_on_cmdlist (list, text, word)
 
   if (matches == 0)
     {
-      free ((PTR)matchlist);
+      free ((PTR) matchlist);
       matchlist = 0;
     }
   else
     {
-      matchlist = (char **) xrealloc ((char *)matchlist, ((matches + 1)
-                                               * sizeof (char *)));
+      matchlist = (char **) xrealloc ((char *) matchlist, ((matches + 1)
+                                                       * sizeof (char *)));
       matchlist[matches] = (char *) 0;
     }
 
@@ -1138,12 +1140,12 @@ complete_on_enum (enumlist, text, word)
        if (matches == sizeof_matchlist)
          {
            sizeof_matchlist *= 2;
-           matchlist = (char **) xrealloc ((char *)matchlist,
+           matchlist = (char **) xrealloc ((char *) matchlist,
                                            (sizeof_matchlist
                                             * sizeof (char *)));
          }
 
-       matchlist[matches] = (char *) 
+       matchlist[matches] = (char *)
          xmalloc (strlen (word) + strlen (name) + 1);
        if (word == text)
          strcpy (matchlist[matches], name);
@@ -1164,13 +1166,13 @@ complete_on_enum (enumlist, text, word)
 
   if (matches == 0)
     {
-      free ((PTR)matchlist);
+      free ((PTR) matchlist);
       matchlist = 0;
     }
   else
     {
-      matchlist = (char **) xrealloc ((char *)matchlist, ((matches + 1)
-                                               * sizeof (char *)));
+      matchlist = (char **) xrealloc ((char *) matchlist, ((matches + 1)
+                                                       * sizeof (char *)));
       matchlist[matches] = (char *) 0;
     }
 
@@ -1195,16 +1197,15 @@ parse_binary_operation (arg)
       || !strncmp (arg, "1", length)
       || !strncmp (arg, "yes", length))
     return 1;
+  else if (!strncmp (arg, "off", length)
+          || !strncmp (arg, "0", length)
+          || !strncmp (arg, "no", length))
+    return 0;
   else
-    if (!strncmp (arg, "off", length)
-       || !strncmp (arg, "0", length)
-       || !strncmp (arg, "no", length))
+    {
+      error ("\"on\" or \"off\" expected.");
       return 0;
-    else 
-      {
-       error ("\"on\" or \"off\" expected.");
-       return 0;
-      }
+    }
 }
 
 /* Do a "set" or "show" command.  ARG is NULL if no argument, or the text
@@ -1227,11 +1228,12 @@ do_setshow_command (arg, from_tty, c)
            char *p;
            char *q;
            int ch;
-           
+
            if (arg == NULL)
              arg = "";
            new = (char *) xmalloc (strlen (arg) + 2);
-           p = arg; q = new;
+           p = arg;
+           q = new;
            while ((ch = *p++) != '\000')
              {
                if (ch == '\\')
@@ -1247,7 +1249,7 @@ do_setshow_command (arg, from_tty, c)
                      break;
                    ch = parse_escape (&p);
                    if (ch == 0)
-                     break; /* C loses */
+                     break;    /* C loses */
                    else if (ch > 0)
                      *q++ = ch;
                  }
@@ -1260,24 +1262,24 @@ do_setshow_command (arg, from_tty, c)
 #endif
            *q++ = '\0';
            new = (char *) xrealloc (new, q - new);
-           if (*(char **)c->var != NULL)
-             free (*(char **)c->var);
+           if (*(char **) c->var != NULL)
+             free (*(char **) c->var);
            *(char **) c->var = new;
          }
          break;
        case var_string_noescape:
          if (arg == NULL)
            arg = "";
-         if (*(char **)c->var != NULL)
-           free (*(char **)c->var);
+         if (*(char **) c->var != NULL)
+           free (*(char **) c->var);
          *(char **) c->var = savestring (arg, strlen (arg));
          break;
        case var_filename:
          if (arg == NULL)
            error_no_arg ("filename to set it to.");
-         if (*(char **)c->var != NULL)
-           free (*(char **)c->var);
-         *(char **)c->var = tilde_expand (arg);
+         if (*(char **) c->var != NULL)
+           free (*(char **) c->var);
+         *(char **) c->var = tilde_expand (arg);
          break;
        case var_boolean:
          *(int *) c->var = parse_binary_operation (arg);
@@ -1332,7 +1334,7 @@ do_setshow_command (arg, from_tty, c)
              }
 
            p = strchr (arg, ' ');
-           
+
            if (p)
              len = p - arg;
            else
@@ -1352,7 +1354,7 @@ do_setshow_command (arg, from_tty, c)
            if (nmatches > 1)
              error ("Ambiguous item \"%s\".", arg);
 
-           *(char **)c->var = match;
+           *(char **) c->var = match;
          }
          break;
        default:
@@ -1363,54 +1365,55 @@ do_setshow_command (arg, from_tty, c)
     {
       /* Print doc minus "show" at start.  */
       print_doc_line (gdb_stdout, c->doc + 5);
-      
+
       fputs_filtered (" is ", gdb_stdout);
       wrap_here ("    ");
       switch (c->var_type)
        {
-      case var_string:
-       {
-         unsigned char *p;
+       case var_string:
+         {
+           unsigned char *p;
 
+           fputs_filtered ("\"", gdb_stdout);
+           if (*(unsigned char **) c->var)
+             for (p = *(unsigned char **) c->var; *p != '\0'; p++)
+               gdb_printchar (*p, gdb_stdout, '"');
+           fputs_filtered ("\"", gdb_stdout);
+         }
+         break;
+       case var_string_noescape:
+       case var_filename:
+       case var_enum:
          fputs_filtered ("\"", gdb_stdout);
-         if (*(unsigned char **)c->var)
-           for (p = *(unsigned char **) c->var; *p != '\0'; p++)
-             gdb_printchar (*p, gdb_stdout, '"');
+         if (*(char **) c->var)
+           fputs_filtered (*(char **) c->var, gdb_stdout);
          fputs_filtered ("\"", gdb_stdout);
-       }
-       break;
-      case var_string_noescape:
-      case var_filename:
-      case var_enum:
-       fputs_filtered ("\"", gdb_stdout);
-       if (*(char **)c->var)
-         fputs_filtered (*(char **) c->var, gdb_stdout);
-       fputs_filtered ("\"", gdb_stdout);
-       break;
-      case var_boolean:
-       fputs_filtered (*(int *) c->var ? "on" : "off", gdb_stdout);
-       break;
-      case var_uinteger:
-       if (*(unsigned int *) c->var == UINT_MAX) {
-         fputs_filtered ("unlimited", gdb_stdout);
          break;
+       case var_boolean:
+         fputs_filtered (*(int *) c->var ? "on" : "off", gdb_stdout);
+         break;
+       case var_uinteger:
+         if (*(unsigned int *) c->var == UINT_MAX)
+           {
+             fputs_filtered ("unlimited", gdb_stdout);
+             break;
+           }
+         /* else fall through */
+       case var_zinteger:
+         fprintf_filtered (gdb_stdout, "%u", *(unsigned int *) c->var);
+         break;
+       case var_integer:
+         if (*(int *) c->var == INT_MAX)
+           {
+             fputs_filtered ("unlimited", gdb_stdout);
+           }
+         else
+           fprintf_filtered (gdb_stdout, "%d", *(int *) c->var);
+         break;
+
+       default:
+         error ("gdb internal error: bad var_type in do_setshow_command");
        }
-       /* else fall through */
-      case var_zinteger:
-       fprintf_filtered (gdb_stdout, "%u", *(unsigned int *) c->var);
-       break;
-      case var_integer:
-       if (*(int *) c->var == INT_MAX)
-         {
-           fputs_filtered ("unlimited", gdb_stdout);
-         }
-       else
-         fprintf_filtered (gdb_stdout, "%d", *(int *) c->var);
-       break;
-           
-      default:
-       error ("gdb internal error: bad var_type in do_setshow_command");
-      }
       fputs_filtered (".\n", gdb_stdout);
     }
   else
@@ -1426,19 +1429,20 @@ cmd_show_list (list, from_tty, prefix)
      int from_tty;
      char *prefix;
 {
-  for (; list != NULL; list = list->next) {
-    /* If we find a prefix, run its list, prefixing our output by its
-       prefix (with "show " skipped).  */
-    if (list->prefixlist && !list->abbrev_flag)
-      cmd_show_list (*list->prefixlist, from_tty, list->prefixname + 5);
-    if (list->type == show_cmd)
-      {
-       fputs_filtered (prefix, gdb_stdout);
-       fputs_filtered (list->name, gdb_stdout);
-       fputs_filtered (":  ", gdb_stdout);
-       do_setshow_command ((char *)NULL, from_tty, list);
-      }
-  }
+  for (; list != NULL; list = list->next)
+    {
+      /* If we find a prefix, run its list, prefixing our output by its
+         prefix (with "show " skipped).  */
+      if (list->prefixlist && !list->abbrev_flag)
+       cmd_show_list (*list->prefixlist, from_tty, list->prefixname + 5);
+      if (list->type == show_cmd)
+       {
+         fputs_filtered (prefix, gdb_stdout);
+         fputs_filtered (list->name, gdb_stdout);
+         fputs_filtered (":  ", gdb_stdout);
+         do_setshow_command ((char *) NULL, from_tty, list);
+       }
+    }
 }
 
 /* ARGSUSED */
@@ -1464,7 +1468,7 @@ shell_escape (arg, from_tty)
   else
     p++;                       /* Get past '/' */
 
-  if ((pid = fork()) == 0)
+  if ((pid = fork ()) == 0)
     {
       if (!arg)
        execl (user_shell, p, 0);
@@ -1496,11 +1500,11 @@ make_command (arg, from_tty)
     p = "make";
   else
     {
-      p = xmalloc (sizeof("make ") + strlen(arg));
+      p = xmalloc (sizeof ("make ") + strlen (arg));
       strcpy (p, "make ");
-      strcpy (p + sizeof("make ")-1, arg);
+      strcpy (p + sizeof ("make ") - 1, arg);
     }
-  
+
   shell_escape (p, from_tty);
 }
 
@@ -1560,11 +1564,11 @@ _initialize_command ()
 With no arguments, run an inferior shell.");
 
   if (xdb_commands)
-    add_com_alias("!", "shell", class_support, 0);
+    add_com_alias ("!", "shell", class_support, 0);
 
   add_com ("make", class_support, make_command,
-          "Run the ``make'' program using the rest of the line as arguments.");
-  add_cmd ("user", no_class, show_user, 
+       "Run the ``make'' program using the rest of the line as arguments.");
+  add_cmd ("user", no_class, show_user,
           "Show definitions of user defined commands.\n\
 Argument is the name of the user defined command.\n\
 With no argument, show definitions of all user defined commands.", &showlist);