[PATCH,HURD] hurd: compliance fixes for getgroups

Pino Toscano toscano.pino@tiscali.it
Sat Apr 28 11:22:00 GMT 2012


Alle venerdì 27 aprile 2012, Roland McGrath ha scritto:
> > 2012-04-27  Pino Toscano  <toscano.pino@tiscali.it>
> > 
> > 	* sysdeps/mach/hurd/getgroups.c (__getgroups): Return -1 and set
> > 	EINVAL for negative `n' or less than `ngids'.
> 
> The norm is to use all caps and no quotes to mention local variable
> names.  You can also drop the (function) when it's the only function
> in the file of the same name.

Ah ok, fixed patch attached.
Where can I read about these two change log style settings (i.e. 
variables as all caps, and no (function) when it's the only one in a 
file)? I don't remember them in the official pages[1], so maybe I missed 
some documentation...

[1] http://www.gnu.org/prep/standards/html_node/Change-Logs.html

> Since I see you already have
> copyright paperwork and you're doing more than just the one change,
> would you like to get set up to commit yourself?

Given also my contributions so far, I guess that I should better need 
review for now, no?

-- 
Pino Toscano
-------------- next part --------------
A non-text attachment was scrubbed...
Name: hurd_getgroups.diff
Type: text/x-patch
Size: 844 bytes
Desc: not available
URL: <http://sourceware.org/pipermail/libc-alpha/attachments/20120428/c9debb68/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 190 bytes
Desc: This is a digitally signed message part.
URL: <http://sourceware.org/pipermail/libc-alpha/attachments/20120428/c9debb68/attachment.sig>


More information about the Libc-alpha mailing list