Re: [PATCH] Unprotected imsg header

2010-06-23 Thread Nicholas Marriott
Hi Adding the multiple inclusion stuff is right, thanks. I don't think we need to include sys/queue.h in imsg.h - the man page documents that you should include sys/types.h and sys/queue.h yourself. On Tue, Jun 22, 2010 at 10:40:45PM -0300, Christiano F. Haesbaert wrote: > Also a missing includ

Re: [PATCH] Unprotected imsg header

2010-06-22 Thread Christiano F. Haesbaert
Also a missing include. Index: imsg.h === RCS file: /cvs/src/lib/libutil/imsg.h,v retrieving revision 1.1 diff -d -u -p -w imsg.h --- imsg.h 26 May 2010 16:44:32 - 1.1 +++ imsg.h 23 Jun 2010 01:36:28 - @@ -18,6

[PATCH] Unprotected imsg header

2010-06-22 Thread Christiano F. Haesbaert
Hi, imsg.h in libutil was unprotected. Index: imsg.h === RCS file: /cvs/src/lib/libutil/imsg.h,v retrieving revision 1.1 diff -d -u -p -w imsg.h --- imsg.h 26 May 2010 16:44:32 - 1.1 +++ imsg.h 23 Jun 2010 01:23:00