Date: Thu, 12 Oct 2006 23:48:00 -0400 From: Tom Rhodes <trhodes@FreeBSD.org> To: Mikhail Teterin <mi+mx@aldan.algebra.com> Cc: trhodes@FreeBSD.org, freebsd-doc@FreeBSD.org, mi@aldan.algebra.com, jmacd@cs.berkeley.edu Subject: Re: docs/65530: [patch] minor improvement to getgrent(3) Message-ID: <20061012234800.3131bd97.trhodes@FreeBSD.org> In-Reply-To: <200610122226.39844.mi%2Bmx@aldan.algebra.com> References: <200610120845.k9C8jDdc070356@freefall.freebsd.org> <200610122226.39844.mi%2Bmx@aldan.algebra.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On Thu, 12 Oct 2006 22:26:39 -0400 Mikhail Teterin <mi+mx@aldan.algebra.com> 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
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20061012234800.3131bd97.trhodes>