Re: [Acl-devel] typo in acl_get_perm?

[Corinna Vinschen]

On Dec 29 01:05, Mike Frysinger wrote:
> On 27 Dec 2015 23:23, Andreas Grünbacher wrote:
> > 2015-12-27 20:51 GMT+01:00 Corinna Vinschen <address@hidden>:
> > > It's not clear from the man page that only a
> > > single permission bit is allowed here, rather than an arbitrary
> > > combination of the above bits.
> > 
> > The acl_add_perm, acl_delete_perm, and acl_get_perm man pages all
> > refer to a permission (singular) for the acl_perm_t argument. This
> > seems to correctly describe the interface, it is not overly verbose
> > though. Maybe we can make it harder to misread things.
> 
> i think confusion arises from the word "permission".  colloquially, i
> think it's accepted to say things like "what is the file permission",
> or "do you have permission [to read or execute]?".
> 
> maybe 

It might help to add something like "perm may be one of the acl_per_t
values ACL_READ, ACL_WRITE, or ACL_EXECUTE".  This is probably redundant
for people being exactly sure how to use these functions but it might
make it clearer for POSIX ACL newbies.


Corinna

signature.asc
Description: PGP signature