Re: [PATCH] Replace ../include/wait.h with gdb_wait.h.

[Michael Snyder <msnyder at cygnus dot com>]

Mark Kettenis wrote:

>    [snip]

> There is a problem with `linux-thread.c', where `gdb_wait.h' is included
> before config.h.  This means that HAVE_SYS_WAIT_H and HAVE_WAIT_H are
> still undefined and the system headers are never used.
> 
> Since `linux-thread.c' uses __W_STOPCODE, and `gdb_wait.h' doesn't
> provide a fallback macro I get a linker failure.  Moving up the
> include for `defs.h' solves this problem.

As noted earlier, the above part of this patch has already
been checked in...
 
> The rest of GDB seems to be using WSETSTOP, so it seems appropriate to
> use that macro instead of __W_STOPCODE in `linux-thread.c'.  I changed
> the definition of WSETSTOP and WSETEXIT in `gdb_wait.h' to use
> W_STOPCODE and W_EXITCODE if they are available.  All BSD-derived
> systems and systems that try to be source-compatible with BSD (like
> Linux and the Hurd) should have those macros.  The change is merely
> cosmetic but there might be systems out there that really use a
> different way to store this information.

And I've now checked in this part as well.

				Michael Snyder