Re: acl: refactor error messages in copy-acl.c

[Bruno Haible]

Jim Meyering wrote:
> >     * lib/copy-acl.c (qcopy_acl): New function, extracted from copy_acl.
> >     (copy_acl): Just a wrapper around qcopy_acl that emits the error
> >     messages.
> 
> Obviously an improvement, and looks correct.
> Go ahead.  Thanks!

Applied, together with a corresponding comment refinement:

        * lib/set-mode-acl.c (qset_acl): Document return value precisely.

--- lib/set-mode-acl.c.orig     2008-06-03 00:04:45.000000000 +0200
+++ lib/set-mode-acl.c  2008-06-03 00:04:16.000000000 +0200
@@ -42,8 +42,8 @@
    filename based operations on NAME.  If access control lists are not
    available, fchmod the target file to MODE.  Also sets the
    non-permission bits of the destination file (S_ISUID, S_ISGID, S_ISVTX)
-   to those from MODE if any are set.  System call return value
-   semantics.  */
+   to those from MODE if any are set.
+   Return 0 if successful.  Return -1 and set errno upon failure.  */
 
 int
 qset_acl (char const *name, int desc, mode_t mode)