diff mbox

New ARI warning Tue Apr 28 01:54:12 UTC 2015

Message ID 4578472.FfTsiG9exU@ralph.baldwin.cx
State New
Headers show

Commit Message

John Baldwin April 28, 2015, 3:33 p.m. UTC 
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

Joel Brobecker April 28, 2015, 3:47 p.m. UTC | #1 
> 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!
diff mbox

Patch

diff --git a/gdb/fbsd-nat.c b/gdb/fbsd-nat.c
index 066e288..9705d45 100644
--- a/gdb/fbsd-nat.c
+++ b/gdb/fbsd-nat.c
@@ -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>