Garance A Drosihn <[EMAIL PROTECTED]> writes:
> At 9:25 PM +0300 3/7/02, Andrey A. Chernov wrote:
> ><grp.h> fix is incomplete because 'u_int32_t' is not defined
> >(but must be) for standalone <grp.h>. Please add some includes
> >for u_int32_t definition too, probably <sys/types.h>

Perhaps subconsciously I was trying to keep making software include
<sys/types.h> before <grp.h>.  :)  I just committed a fix.

> As a minor side question, should we also have that defined as
> uint32_t instead of u_int32_t ?

Yes, usually.  I just grabbed the copy from <sys/types.h>, which
wasn't correct.  uint32_t is more correct, but requires pollution from
another header, so I used the __uint32_t variant.

Best regards,
Mike Barcroft

To Unsubscribe: send mail to [EMAIL PROTECTED]
with "unsubscribe freebsd-current" in the body of the message

Reply via email to