2010-12-28 Michael Snyder <msnyder@vmware.com>

	* event-loop.c: Comment clean-up.
	* event-loop.h: Ditto.
	* event-top.c: Ditto.
	* gdb.c: Ditto.
	* gdb.h: Ditto.
	* main.c: Ditto.
	* top.c: Ditto.
	* top.h: Ditto.

1 files changed, 151 insertions, 146 deletions

diff --git a/gdb/event-top.c b/gdb/event-top.c
index a49bddb..01d53cf 100644
--- a/gdb/event-top.c
+++ b/gdb/event-top.c
@@ -18,7 +18,7 @@
    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, see <http://www.gnu.org/licenses/>. */
+   along with this program.  If not, see <http://www.gnu.org/licenses/>.  */
 
 #include "defs.h"
 #include "top.h"
@@ -34,10 +34,9 @@
 #include "main.h"
 #include "gdbthread.h"
 
-/* For dont_repeat() */
-#include "gdbcmd.h"
+#include "gdbcmd.h"		/* for dont_repeat() */
 
-/* readline include files */
+/* readline include files.  */
 #include "readline/readline.h"
 #include "readline/history.h"
 
@@ -50,7 +49,7 @@ static void change_line_handler (void);
 static void change_annotation_level (void);
 static void command_handler (char *command);
 
-/* Signal handlers. */
+/* Signal handlers.  */
 #ifdef SIGQUIT
 static void handle_sigquit (int sig);
 #endif
@@ -63,7 +62,7 @@ static void handle_sigwinch (int sig);
 #endif
 
 /* Functions to be invoked by the event loop in response to
-   signals. */
+   signals.  */
 #if defined (SIGQUIT) || defined (SIGHUP)
 static void async_do_nothing (gdb_client_data);
 #endif
@@ -76,65 +75,65 @@ static void async_stop_sig (gdb_client_data);
 #endif
 
 /* Readline offers an alternate interface, via callback
-   functions. These are all included in the file callback.c in the
+   functions.  These are all included in the file callback.c in the
    readline distribution.  This file provides (mainly) a function, which
    the event loop uses as callback (i.e. event handler) whenever an event
    is detected on the standard input file descriptor.
    readline_callback_read_char is called (by the GDB event loop) whenever
-   there is a new character ready on the input stream. This function
+   there is a new character ready on the input stream.  This function
    incrementally builds a buffer internal to readline where it
    accumulates the line read up to the point of invocation.  In the
    special case in which the character read is newline, the function
    invokes a GDB supplied callback routine, which does the processing of
    a full command line.  This latter routine is the asynchronous analog
-   of the old command_line_input in gdb. Instead of invoking (and waiting
+   of the old command_line_input in gdb.  Instead of invoking (and waiting
    for) readline to read the command line and pass it back to
    command_loop for processing, the new command_line_handler function has
    the command line already available as its parameter.  INPUT_HANDLER is
    to be set to the function that readline will invoke when a complete
    line of input is ready.  CALL_READLINE is to be set to the function
-   that readline offers as callback to the event_loop. */
+   that readline offers as callback to the event_loop.  */
 
 void (*input_handler) (char *);
 void (*call_readline) (gdb_client_data);
 
-/* Important variables for the event loop. */
+/* Important variables for the event loop.  */
 
 /* This is used to determine if GDB is using the readline library or
-   its own simplified form of readline. It is used by the asynchronous
+   its own simplified form of readline.  It is used by the asynchronous
    form of the set editing command.
    ezannoni: as of 1999-04-29 I expect that this
    variable will not be used after gdb is changed to use the event
-   loop as default engine, and event-top.c is merged into top.c. */
+   loop as default engine, and event-top.c is merged into top.c.  */
 int async_command_editing_p;
 
 /* This variable contains the new prompt that the user sets with the
-   set prompt command. */
+   set prompt command.  */
 char *new_async_prompt;
 
 /* This is the annotation suffix that will be used when the
-   annotation_level is 2. */
+   annotation_level is 2.  */
 char *async_annotation_suffix;
 
 /* This is used to display the notification of the completion of an
-   asynchronous execution command. */
+   asynchronous execution command.  */
 int exec_done_display_p = 0;
 
 /* This is the file descriptor for the input stream that GDB uses to
-   read commands from. */
+   read commands from.  */
 int input_fd;
 
-/* This is the prompt stack. Prompts will be pushed on the stack as
+/* This is the prompt stack.  Prompts will be pushed on the stack as
    needed by the different 'kinds' of user inputs GDB is asking
-   for. See event-loop.h. */
+   for.  See event-loop.h.  */
 struct prompts the_prompts;
 
-/* signal handling variables */
+/* Signal handling variables.  */
 /* Each of these is a pointer to a function that the event loop will
-   invoke if the corresponding signal has received. The real signal
+   invoke if the corresponding signal has received.  The real signal
    handlers mark these functions as ready to be executed and the event
-   loop, in a later iteration, calls them. See the function
-   invoke_async_signal_handler. */
+   loop, in a later iteration, calls them.  See the function
+   invoke_async_signal_handler.  */
 void *sigint_token;
 #ifdef SIGHUP
 void *sighup_token;
@@ -151,10 +150,10 @@ void *sigtstp_token;
 #endif
 
 /* Structure to save a partially entered command.  This is used when
-   the user types '\' at the end of a command line. This is necessary
+   the user types '\' at the end of a command line.  This is necessary
    because each line of input is handled by a different call to
    command_line_handler, and normally there is no state retained
-   between different calls. */
+   between different calls.  */
 int more_to_come = 0;
 
 struct readline_input_state
@@ -169,9 +168,9 @@ readline_input_state;
 void (*after_char_processing_hook) ();
 
 
-/* Wrapper function for calling into the readline library. The event
-   loop expects the callback function to have a paramter, while readline 
-   expects none. */
+/* Wrapper function for calling into the readline library.  The event
+   loop expects the callback function to have a paramter, while
+   readline expects none.  */
 static void
 rl_callback_read_char_wrapper (gdb_client_data client_data)
 {
@@ -181,21 +180,21 @@ rl_callback_read_char_wrapper (gdb_client_data client_data)
 }
 
 /* Initialize all the necessary variables, start the event loop,
-   register readline, and stdin, start the loop. */
+   register readline, and stdin, start the loop.  */
 void
 cli_command_loop (void)
 {
   /* If we are using readline, set things up and display the first
-     prompt, otherwise just print the prompt. */
+     prompt, otherwise just print the prompt.  */
   if (async_command_editing_p)
     {
       int length;
       char *a_prompt;
       char *gdb_prompt = get_prompt ();
 
-      /* Tell readline what the prompt to display is and what function it
-         will need to call after a whole line is read. This also displays
-         the first prompt. */
+      /* Tell readline what the prompt to display is and what function
+         it will need to call after a whole line is read.  This also
+         displays the first prompt.  */
       length = strlen (PREFIX (0)) 
 	+ strlen (gdb_prompt) + strlen (SUFFIX (0)) + 1;
       a_prompt = (char *) alloca (length);
@@ -207,54 +206,54 @@ cli_command_loop (void)
   else
     display_gdb_prompt (0);
 
-  /* Now it's time to start the event loop. */
+  /* Now it's time to start the event loop.  */
   start_event_loop ();
 }
 
 /* Change the function to be invoked every time there is a character
-   ready on stdin. This is used when the user sets the editing off,
+   ready on stdin.  This is used when the user sets the editing off,
    therefore bypassing readline, and letting gdb handle the input
-   itself, via gdb_readline2. Also it is used in the opposite case in
+   itself, via gdb_readline2.  Also it is used in the opposite case in
    which the user sets editing on again, by restoring readline
-   handling of the input. */
+   handling of the input.  */
 static void
 change_line_handler (void)
 {
-  /* NOTE: this operates on input_fd, not instream. If we are reading
-     commands from a file, instream will point to the file. However in
+  /* NOTE: this operates on input_fd, not instream.  If we are reading
+     commands from a file, instream will point to the file.  However in
      async mode, we always read commands from a file with editing
-     off. This means that the 'set editing on/off' will have effect
-     only on the interactive session. */
+     off.  This means that the 'set editing on/off' will have effect
+     only on the interactive session.  */
 
   if (async_command_editing_p)
     {
-      /* Turn on editing by using readline. */
+      /* Turn on editing by using readline.  */
       call_readline = rl_callback_read_char_wrapper;
       input_handler = command_line_handler;
     }
   else
     {
-      /* Turn off editing by using gdb_readline2. */
+      /* Turn off editing by using gdb_readline2.  */
       rl_callback_handler_remove ();
       call_readline = gdb_readline2;
 
       /* Set up the command handler as well, in case we are called as
-         first thing from .gdbinit. */
+         first thing from .gdbinit.  */
       input_handler = command_line_handler;
     }
 }
 
 /* Displays the prompt. The prompt that is displayed is the current
    top of the prompt stack, if the argument NEW_PROMPT is
-   0. Otherwise, it displays whatever NEW_PROMPT is. This is used
+   0. Otherwise, it displays whatever NEW_PROMPT is.  This is used
    after each gdb command has completed, and in the following cases:
-   1. when the user enters a command line which is ended by '\'
+   1. When the user enters a command line which is ended by '\'
    indicating that the command will continue on the next line.
    In that case the prompt that is displayed is the empty string.
    2. When the user is entering 'commands' for a breakpoint, or
-   actions for a tracepoint. In this case the prompt will be '>'
+   actions for a tracepoint.  In this case the prompt will be '>'
    3. Other????
-   FIXME: 2. & 3. not implemented yet for async. */
+   FIXME: 2. & 3. not implemented yet for async.  */
 void
 display_gdb_prompt (char *new_prompt)
 {
@@ -276,15 +275,15 @@ display_gdb_prompt (char *new_prompt)
          function, readline still tries to do its own display if we
          don't call rl_callback_handler_install and
          rl_callback_handler_remove (which readline detects because a
-         global variable is not set). If readline did that, it could
+         global variable is not set).  If readline did that, it could
          mess up gdb signal handlers for SIGINT.  Readline assumes
          that between calls to rl_set_signals and rl_clear_signals gdb
-         doesn't do anything with the signal handlers. Well, that's
+         doesn't do anything with the signal handlers.  Well, that's
          not the case, because when the target executes we change the
-         SIGINT signal handler. If we allowed readline to display the
+         SIGINT signal handler.  If we allowed readline to display the
          prompt, the signal handler change would happen exactly
          between the calls to the above two functions.
-         Calling rl_callback_handler_remove(), does the job. */
+         Calling rl_callback_handler_remove(), does the job.  */
 
       rl_callback_handler_remove ();
       return;
@@ -292,18 +291,18 @@ display_gdb_prompt (char *new_prompt)
 
   if (!new_prompt)
     {
-      /* Just use the top of the prompt stack. */
+      /* Just use the top of the prompt stack.  */
       prompt_length = strlen (PREFIX (0)) +
 	strlen (SUFFIX (0)) +
 	strlen (gdb_prompt) + 1;
 
       new_prompt = (char *) alloca (prompt_length);
 
-      /* Prefix needs to have new line at end. */
+      /* Prefix needs to have new line at end.  */
       strcpy (new_prompt, PREFIX (0));
       strcat (new_prompt, gdb_prompt);
       /* Suffix needs to have a new line at end and \032 \032 at
-         beginning. */
+         beginning.  */
       strcat (new_prompt, SUFFIX (0));
     }
 
@@ -312,7 +311,8 @@ display_gdb_prompt (char *new_prompt)
       rl_callback_handler_remove ();
       rl_callback_handler_install (new_prompt, input_handler);
     }
-  /* new_prompt at this point can be the top of the stack or the one passed in */
+  /* new_prompt at this point can be the top of the stack or the one
+     passed in.  */
   else if (new_prompt)
     {
       /* Don't use a _filtered function here.  It causes the assumed
@@ -324,10 +324,10 @@ display_gdb_prompt (char *new_prompt)
 }
 
 /* Used when the user requests a different annotation level, with
-   'set annotate'. It pushes a new prompt (with prefix and suffix) on top
+   'set annotate'.  It pushes a new prompt (with prefix and suffix) on top
    of the prompt stack, if the annotation level desired is 2, otherwise
    it pops the top of the prompt stack when we want the annotation level
-   to be the normal ones (1 or 0). */
+   to be the normal ones (1 or 0).  */
 static void
 change_annotation_level (void)
 {
@@ -336,7 +336,7 @@ change_annotation_level (void)
   if (!PREFIX (0) || !PROMPT (0) || !SUFFIX (0))
     {
       /* The prompt stack has not been initialized to "", we are
-         using gdb w/o the --async switch */
+         using gdb w/o the --async switch.  */
       warning (_("Command has same effect as set annotate"));
       return;
     }
@@ -345,7 +345,7 @@ change_annotation_level (void)
     {
       if (!strcmp (PREFIX (0), "") && !strcmp (SUFFIX (0), ""))
 	{
-	  /* Push a new prompt if the previous annotation_level was not >1. */
+	  /* Push a new prompt if the previous annotation_level was not >1.  */
 	  prefix = (char *) alloca (strlen (async_annotation_suffix) + 10);
 	  strcpy (prefix, "\n\032\032pre-");
 	  strcat (prefix, async_annotation_suffix);
@@ -363,16 +363,16 @@ change_annotation_level (void)
     {
       if (strcmp (PREFIX (0), "") && strcmp (SUFFIX (0), ""))
 	{
-	  /* Pop the top of the stack, we are going back to annotation < 1. */
+	  /* Pop the top of the stack, we are going back to annotation < 1.  */
 	  pop_prompt ();
 	}
     }
 }
 
-/* Pushes a new prompt on the prompt stack. Each prompt has three
-   parts: prefix, prompt, suffix. Usually prefix and suffix are empty
-   strings, except when the annotation level is 2. Memory is allocated
-   within xstrdup for the new prompt. */
+/* Pushes a new prompt on the prompt stack.  Each prompt has three
+   parts: prefix, prompt, suffix.  Usually prefix and suffix are empty
+   strings, except when the annotation level is 2.  Memory is allocated
+   within xstrdup for the new prompt.  */
 void
 push_prompt (char *prefix, char *prompt, char *suffix)
 {
@@ -380,8 +380,8 @@ push_prompt (char *prefix, char *prompt, char *suffix)
   PREFIX (0) = xstrdup (prefix);
 
   /* Note that this function is used by the set annotate 2
-     command. This is why we take care of saving the old prompt
-     in case a new one is not specified. */
+     command.  This is why we take care of saving the old prompt
+     in case a new one is not specified.  */
   if (prompt)
     PROMPT (0) = xstrdup (prompt);
   else
@@ -390,17 +390,18 @@ push_prompt (char *prefix, char *prompt, char *suffix)
   SUFFIX (0) = xstrdup (suffix);
 }
 
-/* Pops the top of the prompt stack, and frees the memory allocated for it. */
+/* Pops the top of the prompt stack, and frees the memory allocated
+   for it.  */
 void
 pop_prompt (void)
 {
   /* If we are not during a 'synchronous' execution command, in which
-     case, the top prompt would be empty. */
+     case, the top prompt would be empty.  */
   if (strcmp (PROMPT (0), ""))
     /* This is for the case in which the prompt is set while the
-       annotation level is 2. The top prompt will be changed, but when
+       annotation level is 2.  The top prompt will be changed, but when
        we return to annotation level < 2, we want that new prompt to be
-       in effect, until the user does another 'set prompt'. */
+       in effect, until the user does another 'set prompt'.  */
     if (strcmp (PROMPT (0), PROMPT (-1)))
       {
 	xfree (PROMPT (-1));
@@ -416,7 +417,7 @@ pop_prompt (void)
 /* When there is an event ready on the stdin file desriptor, instead
    of calling readline directly throught the callback function, or
    instead of calling gdb_readline2, give gdb a chance to detect
-   errors and do something. */
+   errors and do something.  */
 void
 stdin_event_handler (int error, gdb_client_data client_data)
 {
@@ -426,7 +427,7 @@ stdin_event_handler (int error, gdb_client_data client_data)
       delete_file_handler (input_fd);
       discard_all_continuations ();
       discard_all_intermediate_continuations ();
-      /* If stdin died, we may as well kill gdb. */
+      /* If stdin died, we may as well kill gdb.  */
       quit_command ((char *) 0, stdin == instream);
     }
   else
@@ -435,17 +436,17 @@ stdin_event_handler (int error, gdb_client_data client_data)
 
 /* Re-enable stdin after the end of an execution command in
    synchronous mode, or after an error from the target, and we aborted
-   the exec operation. */
+   the exec operation.  */
 
 void
 async_enable_stdin (void)
 {
   if (sync_execution)
     {
-      /* See NOTE in async_disable_stdin() */
+      /* See NOTE in async_disable_stdin().  */
       /* FIXME: cagney/1999-09-27: Call this before clearing
 	 sync_execution.  Current target_terminal_ours() implementations
-	 check for sync_execution before switching the terminal. */
+	 check for sync_execution before switching the terminal.  */
       target_terminal_ours ();
       pop_prompt ();
       sync_execution = 0;
@@ -453,7 +454,7 @@ async_enable_stdin (void)
 }
 
 /* Disable reads from stdin (the console) marking the command as
-   synchronous. */
+   synchronous.  */
 
 void
 async_disable_stdin (void)
@@ -466,12 +467,12 @@ async_disable_stdin (void)
 }
 
 
-/* Handles a gdb command. This function is called by
+/* Handles a gdb command.  This function is called by
    command_line_handler, which has processed one or more input lines
-   into COMMAND. */
+   into COMMAND.  */
 /* NOTE: 1999-04-30 This is the asynchronous version of the command_loop
    function.  The command_loop function will be obsolete when we
-   switch to use the event loop at every execution of gdb. */
+   switch to use the event loop at every execution of gdb.  */
 static void
 command_handler (char *command)
 {
@@ -482,11 +483,11 @@ command_handler (char *command)
   if (instream == stdin && stdin_is_tty)
     reinitialize_more_filter ();
 
-  /* If readline returned a NULL command, it means that the 
-     connection with the terminal is gone. This happens at the
-     end of a testsuite run, after Expect has hung up 
-     but GDB is still alive. In such a case, we just quit gdb
-     killing the inferior program too. */
+  /* If readline returned a NULL command, it means that the connection
+     with the terminal is gone.  This happens at the end of a
+     testsuite run, after Expect has hung up but GDB is still alive.
+     In such a case, we just quit gdb killing the inferior program
+     too.  */
   if (command == 0)
     {
       printf_unfiltered ("quit\n");
@@ -503,14 +504,15 @@ command_handler (char *command)
   do_cleanups (stat_chain);
 }
 
-/* Handle a complete line of input. This is called by the callback
-   mechanism within the readline library.  Deal with incomplete commands
-   as well, by saving the partial input in a global buffer.  */
+/* Handle a complete line of input.  This is called by the callback
+   mechanism within the readline library.  Deal with incomplete
+   commands as well, by saving the partial input in a global
+   buffer.  */
 
 /* NOTE: 1999-04-30 This is the asynchronous version of the
-   command_line_input function. command_line_input will become
+   command_line_input function; command_line_input will become
    obsolete once we use the event loop as the default mechanism in
-   GDB. */
+   GDB.  */
 static void
 command_line_handler (char *rl)
 {
@@ -555,7 +557,8 @@ command_line_handler (char *rl)
 #endif
 
   /* Make sure that all output has been output.  Some machines may let
-     you get away with leaving out some of the gdb_flush, but not all.  */
+     you get away with leaving out some of the gdb_flush, but not
+     all.  */
   wrap_here ("");
   gdb_flush (gdb_stdout);
   gdb_flush (gdb_stderr);
@@ -564,12 +567,12 @@ command_line_handler (char *rl)
     ++source_line_number;
 
   /* If we are in this case, then command_handler will call quit 
-     and exit from gdb. */
+     and exit from gdb.  */
   if (!rl || rl == (char *) EOF)
     {
       got_eof = 1;
       command_handler (0);
-      return;			/* Lint. */
+      return;			/* Lint.  */
     }
   if (strlen (rl) + 1 + (p - linebuffer) > linelength)
     {
@@ -580,7 +583,7 @@ command_line_handler (char *rl)
     }
   p1 = rl;
   /* Copy line.  Don't copy null at end.  (Leaves line alone
-     if this was just a newline)  */
+     if this was just a newline).  */
   while (*p1)
     *p++ = *p1++;
 
@@ -595,8 +598,8 @@ command_line_handler (char *rl)
       readline_input_state.linebuffer_ptr = p;
 
       /* We will not invoke a execute_command if there is more
-	 input expected to complete the command. So, we need to
-	 print an empty prompt here. */
+	 input expected to complete the command.  So, we need to
+	 print an empty prompt here.  */
       more_to_come = 1;
       push_prompt ("", "", "");
       display_gdb_prompt (0);
@@ -654,9 +657,8 @@ command_line_handler (char *rl)
       xfree (history_value);
     }
 
-  /* If we just got an empty line, and that is supposed
-     to repeat the previous command, return the value in the
-     global buffer.  */
+  /* If we just got an empty line, and that is supposed to repeat the
+     previous command, return the value in the global buffer.  */
   if (repeat && p == linebuffer && *p != '\\')
     {
       command_handler (line);
@@ -686,7 +688,7 @@ command_line_handler (char *rl)
      and remove the '#'.  The kill ring is probably better, but some
      people are in the habit of commenting things out.  */
   if (*p1 == '#')
-    *p1 = '\0';			/* Found a comment. */
+    *p1 = '\0';			/* Found a comment.  */
 
   /* Save into global buffer if appropriate.  */
   if (repeat)
@@ -711,11 +713,11 @@ command_line_handler (char *rl)
 }
 
 /* Does reading of input from terminal w/o the editing features
-   provided by the readline library. */
+   provided by the readline library.  */
 
-/* NOTE: 1999-04-30 Asynchronous version of gdb_readline. gdb_readline
+/* NOTE: 1999-04-30 Asynchronous version of gdb_readline; gdb_readline
    will become obsolete when the event loop is made the default
-   execution for gdb. */
+   execution for gdb.  */
 void
 gdb_readline2 (gdb_client_data client_data)
 {
@@ -726,11 +728,11 @@ gdb_readline2 (gdb_client_data client_data)
   static int done_once = 0;
 
   /* Unbuffer the input stream, so that, later on, the calls to fgetc
-     fetch only one char at the time from the stream. The fgetc's will
+     fetch only one char at the time from the stream.  The fgetc's will
      get up to the first newline, but there may be more chars in the
-     stream after '\n'. If we buffer the input and fgetc drains the
+     stream after '\n'.  If we buffer the input and fgetc drains the
      stream, getting stuff beyond the newline as well, a select, done
-     afterwards will not trigger. */
+     afterwards will not trigger.  */
   if (!done_once && !ISATTY (instream))
     {
       setbuf (instream, NULL);
@@ -742,9 +744,9 @@ gdb_readline2 (gdb_client_data client_data)
   /* We still need the while loop here, even though it would seem
      obvious to invoke gdb_readline2 at every character entered.  If
      not using the readline library, the terminal is in cooked mode,
-     which sends the characters all at once. Poll will notice that the
-     input fd has changed state only after enter is pressed. At this
-     point we still need to fetch all the chars entered. */
+     which sends the characters all at once.  Poll will notice that the
+     input fd has changed state only after enter is pressed.  At this
+     point we still need to fetch all the chars entered.  */
 
   while (1)
     {
@@ -755,9 +757,9 @@ gdb_readline2 (gdb_client_data client_data)
       if (c == EOF)
 	{
 	  if (input_index > 0)
-	    /* The last line does not end with a newline.  Return it, and
-	       if we are called again fgetc will still return EOF and
-	       we'll return NULL then.  */
+	    /* The last line does not end with a newline.  Return it,
+	       and if we are called again fgetc will still return EOF
+	       and we'll return NULL then.  */
 	    break;
 	  xfree (result);
 	  (*input_handler) (0);
@@ -785,17 +787,17 @@ gdb_readline2 (gdb_client_data client_data)
 
 
 /* Initialization of signal handlers and tokens.  There is a function
-   handle_sig* for each of the signals GDB cares about. Specifically:
+   handle_sig* for each of the signals GDB cares about.  Specifically:
    SIGINT, SIGFPE, SIGQUIT, SIGTSTP, SIGHUP, SIGWINCH.  These
    functions are the actual signal handlers associated to the signals
    via calls to signal().  The only job for these functions is to
    enqueue the appropriate event/procedure with the event loop.  Such
-   procedures are the old signal handlers. The event loop will take
+   procedures are the old signal handlers.  The event loop will take
    care of invoking the queued procedures to perform the usual tasks
-   associated with the reception of the signal. */
+   associated with the reception of the signal.  */
 /* NOTE: 1999-04-30 This is the asynchronous version of init_signals.
    init_signals will become obsolete as we move to have to event loop
-   as the default for gdb. */
+   as the default for gdb.  */
 void
 async_init_signals (void)
 {
@@ -853,8 +855,8 @@ mark_async_signal_handler_wrapper (void *token)
   mark_async_signal_handler ((struct async_signal_handler *) token);
 }
 
-/* Tell the event loop what to do if SIGINT is received. 
-   See event-signal.c. */
+/* Tell the event loop what to do if SIGINT is received.
+   See event-signal.c.  */
 void
 handle_sigint (int sig)
 {
@@ -862,19 +864,19 @@ handle_sigint (int sig)
 
   /* We could be running in a loop reading in symfiles or something so
      it may be quite a while before we get back to the event loop.  So
-     set quit_flag to 1 here. Then if QUIT is called before we get to
+     set quit_flag to 1 here.  Then if QUIT is called before we get to
      the event loop, we will unwind as expected.  */
 
   quit_flag = 1;
 
   /* If immediate_quit is set, we go ahead and process the SIGINT right
-     away, even if we usually would defer this to the event loop. The
+     away, even if we usually would defer this to the event loop.  The
      assumption here is that it is safe to process ^C immediately if
-     immediate_quit is set. If we didn't, SIGINT would be really
+     immediate_quit is set.  If we didn't, SIGINT would be really
      processed only the next time through the event loop.  To get to
      that point, though, the command that we want to interrupt needs to
      finish first, which is unacceptable.  If immediate quit is not set,
-     we process SIGINT the next time through the loop, which is fine. */
+     we process SIGINT the next time through the loop, which is fine.  */
   gdb_call_async_signal_handler (sigint_token, immediate_quit);
 }
 
@@ -887,7 +889,7 @@ handle_sigterm (int sig)
   quit_force ((char *) 0, stdin == instream);
 }
 
-/* Do the quit. All the checks have been done by the caller. */
+/* Do the quit.  All the checks have been done by the caller.  */
 void
 async_request_quit (gdb_client_data arg)
 {
@@ -895,15 +897,15 @@ async_request_quit (gdb_client_data arg)
      back here, that means that an exception was thrown to unwind the
      current command before we got back to the event loop.  So there
      is no reason to call quit again here, unless immediate_quit is
-     set.*/
+     set.  */
 
   if (quit_flag || immediate_quit)
     quit ();
 }
 
 #ifdef SIGQUIT
-/* Tell the event loop what to do if SIGQUIT is received. 
-   See event-signal.c. */
+/* Tell the event loop what to do if SIGQUIT is received.
+   See event-signal.c.  */
 static void
 handle_sigquit (int sig)
 {
@@ -918,13 +920,13 @@ handle_sigquit (int sig)
 static void
 async_do_nothing (gdb_client_data arg)
 {
-  /* Empty function body. */
+  /* Empty function body.  */
 }
 #endif
 
 #ifdef SIGHUP
-/* Tell the event loop what to do if SIGHUP is received. 
-   See event-signal.c. */
+/* Tell the event loop what to do if SIGHUP is received.
+   See event-signal.c.  */
 static void
 handle_sighup (int sig)
 {
@@ -932,14 +934,14 @@ handle_sighup (int sig)
   signal (sig, handle_sighup);
 }
 
-/* Called by the event loop to process a SIGHUP */
+/* Called by the event loop to process a SIGHUP.  */
 static void
 async_disconnect (gdb_client_data arg)
 {
   catch_errors (quit_cover, NULL,
 		"Could not kill the program being debugged",
 		RETURN_MASK_ALL);
-  signal (SIGHUP, SIG_DFL);	/*FIXME: ??????????? */
+  signal (SIGHUP, SIG_DFL);	/*FIXME: ???????????  */
   raise (SIGHUP);
 }
 #endif
@@ -977,13 +979,14 @@ async_stop_sig (gdb_client_data arg)
   printf_unfiltered ("%s", prompt);
   gdb_flush (gdb_stdout);
 
-  /* Forget about any previous command -- null line now will do nothing.  */
+  /* Forget about any previous command -- null line now will do
+     nothing.  */
   dont_repeat ();
 }
 #endif /* STOP_SIGNAL */
 
-/* Tell the event loop what to do if SIGFPE is received. 
-   See event-signal.c. */
+/* Tell the event loop what to do if SIGFPE is received.
+   See event-signal.c.  */
 static void
 handle_sigfpe (int sig)
 {
@@ -991,17 +994,17 @@ handle_sigfpe (int sig)
   signal (sig, handle_sigfpe);
 }
 
-/* Event loop will call this functin to process a SIGFPE. */
+/* Event loop will call this functin to process a SIGFPE.  */
 static void
 async_float_handler (gdb_client_data arg)
 {
-  /* This message is based on ANSI C, section 4.7. Note that integer
-     divide by zero causes this, so "float" is a misnomer. */
+  /* This message is based on ANSI C, section 4.7.  Note that integer
+     divide by zero causes this, so "float" is a misnomer.  */
   error (_("Erroneous arithmetic operation."));
 }
 
-/* Tell the event loop what to do if SIGWINCH is received. 
-   See event-signal.c. */
+/* Tell the event loop what to do if SIGWINCH is received.
+   See event-signal.c.  */
 #if defined(SIGWINCH) && defined(SIGWINCH_HANDLER)
 static void
 handle_sigwinch (int sig)
@@ -1014,14 +1017,16 @@ handle_sigwinch (int sig)
 
 /* Called by do_setshow_command.  */
 void
-set_async_editing_command (char *args, int from_tty, struct cmd_list_element *c)
+set_async_editing_command (char *args, int from_tty,
+			   struct cmd_list_element *c)
 {
   change_line_handler ();
 }
 
 /* Called by do_setshow_command.  */
 void
-set_async_annotation_level (char *args, int from_tty, struct cmd_list_element *c)
+set_async_annotation_level (char *args, int from_tty,
+			    struct cmd_list_element *c)
 {
   change_annotation_level ();
 }
@@ -1035,7 +1040,7 @@ set_async_prompt (char *args, int from_tty, struct cmd_list_element *c)
 
 /* Set things up for readline to be invoked via the alternate
    interface, i.e. via a callback function (rl_callback_read_char),
-   and hook up instream to the event loop. */
+   and hook up instream to the event loop.  */
 void
 gdb_setup_readline (void)
 {
@@ -1054,7 +1059,7 @@ gdb_setup_readline (void)
      editing.  */
   if (ISATTY (instream))
     {
-      /* Tell gdb that we will be using the readline library. This
+      /* Tell gdb that we will be using the readline library.  This
 	 could be overwritten by a command in .gdbinit like 'set
 	 editing on' or 'off'.  */
       async_command_editing_p = 1;
@@ -1070,11 +1075,11 @@ gdb_setup_readline (void)
     }
   
   /* When readline has read an end-of-line character, it passes the
-     complete line to gdb for processing. command_line_handler is the
+     complete line to gdb for processing; command_line_handler is the
      function that does this.  */
   input_handler = command_line_handler;
       
-  /* Tell readline to use the same input stream that gdb uses. */
+  /* Tell readline to use the same input stream that gdb uses.  */
   rl_instream = instream;
 
   /* Get a file descriptor for the input stream, so that we can
@@ -1084,7 +1089,7 @@ gdb_setup_readline (void)
   /* Now we need to create the event sources for the input file
      descriptor.  */
   /* At this point in time, this is the only event source that we
-     register with the even loop. Another source is going to be the
+     register with the even loop.  Another source is going to be the
      target program (inferior), but that must be registered only when
      it actually exists (I.e. after we say 'run' or after we connect
      to a remote target.  */