Re: docs/65530: [patch] minor improvement to getgrent(3)

On Thu, 12 Oct 2006 22:26:39 -0400
Mikhail Teterin <[e-mail]> wrote:

> =DE=C5=D4=D7=C5=D2 12 =D6=CF=D7=D4=C5=CE=D8 2006 04:45, Tom Rhodes =CE=C1=
=D0=C9=D3=C1=D7:
> > This information is duplicated from the /usr/include/grp.h
> > header file, we should change both or none.
>=20
> Why? Header files are only a second-from-last sources of information=20
> (implementation being the last). A good manual page leaves no need to loo=
k at=20
> the declaration.

Then we should patch both, not just one.

>=20
> > I don't see a good reason to really change them right now as it seems s=
eems
> > self-explainatory. =9AThanks for the submission!
>=20
> ?? The information was put there in 1996 -- it just got lost because of a=
 cvs=20
> mishap.
>=20
> I don't believe it to be self-explanatory -- there is not even a single=20
> example in the man-page. One could *guess*, that the list is null termina=
ted,=20
> but it should be explicit, or else time will be wasted searching for othe=
r=20
> api (getgroupnumber() ?) or writing test-programs.

To me it seems fine, but if you want both patched then go for
it.  Send me a patch please.  ;)

--=20
Tom Rhodes
_______________________________________________
[e-mail] mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-doc
To unsubscribe, send any mail to "[e-mail]"