Re: cvs commit: src/include

[Matthew Dillon <dillon@xxxxxxxxxxxxxxxxxxxx>]

:liamfoy     2004/10/31 11:01:00 PST
:
:DragonFly src repository
:
:  Modified files:
:    include              grp.h 
:  Log:
:  The gr_gid member of the struct group should be gid_t
:  and not an int.
:  
:  Submitted by: Douwe Kiela
:  
:  Revision  Changes    Path
:  1.3       +1 -1      src/include/grp.h
:
:
:http://www.dragonflybsd.org/cvsweb/src/include/grp.h.diff?r1=1.2&r2=1.3&f=u

    It's not quite that easy.  You can't just use a typedef type in an
    official header file without also making sure the typedef type itself
    is declared.

    Many of our header files #include <sys/types.h> (e.g. take a look at pwd.h)
    for precisely this reason.  FreeBSD has gone a step further and tried to
    remove all type pollution but, generally speaking, I think we can get
    away with just including sys/types.h for now.  So do that too.

					-Matt
					Matthew Dillon 
					<dillon@xxxxxxxxxxxxx>