New ARI warning Tue Apr 28 01:54:12 UTC 2015
Commit Message
On Tuesday, April 28, 2015 08:31:25 AM Joel Brobecker wrote:
> John,
>
> On Tue, Apr 28, 2015 at 01:54:13AM +0000, GDB Administrator wrote:
> > 169a170
> > > gdb/fbsd-nat.c:30: regression: wait.h: Do not include wait.h or sys/wait.h, instead include gdb_wait.h
> > gdb/fbsd-nat.c:30:#include <sys/wait.h>
>
> Looking at the file that triggered this new warning, I'd guess
> this one comes from one of your changes. It might make a little
> less sense in a -nat file to follow that rule, but I don't think
> it'd hurt, and this would avoid increasing the number of ARI hits.
> Would you mind fixing, please?
Not at all, does this look ok?
Use "gdb_wait.h" instead of <sys/wait.h>.
gdb/ChangeLog:
* fbsd-nat.c: Include "gdb_wait.h" instead of <sys/wait.h>.
---
Comments
> Not at all, does this look ok?
Thanks!
> Use "gdb_wait.h" instead of <sys/wait.h>.
>
> gdb/ChangeLog:
> * fbsd-nat.c: Include "gdb_wait.h" instead of <sys/wait.h>.
Looks good to me!
@@ -23,11 +23,11 @@
#include "regcache.h"
#include "regset.h"
#include "gdbthread.h"
+#include "gdb_wait.h"
#include <sys/types.h>
#include <sys/procfs.h>
#include <sys/ptrace.h>
#include <sys/sysctl.h>
-#include <sys/wait.h>
#ifdef HAVE_KINFO_GETVMMAP
#include <sys/user.h>
#include <libutil.h>