Add comments noting that vfork might be defined to be fork.
authorKevin Buettner <kevinb@redhat.com>
Sun, 8 Apr 2001 18:26:03 +0000 (18:26 +0000)
committerKevin Buettner <kevinb@redhat.com>
Sun, 8 Apr 2001 18:26:03 +0000 (18:26 +0000)
gdb/ChangeLog
gdb/fork-child.c
gdb/ser-pipe.c

index 8085d06..deec137 100644 (file)
@@ -1,5 +1,12 @@
 2001-04-08  Kevin Buettner  <kevinb@redhat.com>
 
+       * fork-child.c (fork_inferior, clone_and_follow_inferior): 
+       Document fact that apparent call to vfork() might actually be
+       a call to fork() instead.
+       * ser-pipe.c (pipe_open): Likewise.
+
+2001-04-08  Kevin Buettner  <kevinb@redhat.com>
+
        * printcmd.c (print_frame_args): Use a cleanup to invoke
        ui_out_list_end() so that the list count nesting flag will
        be decremented properly when an error occurs.
index a952d4f..1c19b83 100644 (file)
@@ -253,6 +253,9 @@ fork_inferior (char *exec_file_arg, char *allargs, char **env,
   if (pre_trace_fun != NULL)
     (*pre_trace_fun) ();
 
+  /* Create the child process.  Note that the apparent call to vfork()
+     below *might* actually be a call to fork() due to the fact that
+     autoconf will ``#define vfork fork'' on certain platforms.  */
   if (debug_fork)
     pid = fork ();
   else
@@ -420,7 +423,9 @@ clone_and_follow_inferior (int child_pid, int *followed_child)
   if (status < 0)
     error ("error getting pipe for handoff semaphore");
 
-  /* Clone the debugger. */
+  /* Clone the debugger.  Note that the apparent call to vfork()
+     below *might* actually be a call to fork() due to the fact that
+     autoconf will ``#define vfork fork'' on certain platforms.  */
   if (debug_fork)
     debugger_pid = fork ();
   else
index 652d774..3e23287 100644 (file)
@@ -65,6 +65,10 @@ pipe_open (serial_t scb, const char *name)
   if (socketpair (AF_UNIX, SOCK_STREAM, 0, pdes) < 0)
     return -1;
 
+  /* Create the child process to run the command in.  Note that the
+     apparent call to vfork() below *might* actually be a call to
+     fork() due to the fact that autoconf will ``#define vfork fork''
+     on certain platforms.  */
   pid = vfork ();
   
   /* Error. */