Re: [patch] Zap more #ifdef HAVE_VFORK

[Andrew Cagney <ac131313@cygnus.com>]

Kevin Buettner wrote:
> 
> On Mar 26, 11:45am, Andrew Cagney wrote:
> 
> > Missed this when re-fixing the autoconfed vfork() call.
> [...]
> >     /* Clone the debugger. */
> > - #ifdef HAVE_VFORK
> >     if (debug_fork)
> >       debugger_pid = fork ();
> >     else
> >       debugger_pid = vfork ();
> > - #else
> > -   debugger_pid = fork ();
> > - #endif
> 
> This didn't make any sense to me at first.  It did when I went back
> and (re)read

I know it doesn't make sense :-)  It is how autoconf does it though :-/

> If we're going to use the AC_FUNC_VFORK mechanisms, might I suggest
> that we do one of the following?
> 
>     1) Document the fact that the autoconf cleverness *might* actually
>        have defined vfork to be fork at each use vfork.

Ok by me.  In general adding comments explaining how bits of code work
are probably obvious fixes.

>     2) Create a gdb_fork() which does the appropriate thing *and*
>        documents the autoconf cleverness in the guts of gdb_fork().

I don't think this one would work very well.  From memory you're not
ment to return from a vfork().

	Andrew