Re: [PATCH,HURD] hurd: compliance fixes for getgroups

[Pino Toscano]

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

hurd_getgroups.diff
Description: Text Data

signature.asc
Description: This is a digitally signed message part.