acl, copy-file: small refactorings

[Bruno Haible]

Here's a proposed series of tweaks and alignments to the 'acl' and 'copy-file'
modules, to make Reuben's proposed change to 'copy-file' easier.

The first change is not backward compatible, but the only users of copy_acl()
are coreutils and 'copy-file' in gnulib, and I've verified that these callers
don't care whether the return value is -1 or -2.

For the third change, I considered exporting just one function copy_acl
with a null_stderr argument, like in the module 'wait-process', rather than
two function qcopy_acl and copy_acl. But I find the approach with two
functions more maintainable: It would make it easier to move the code
that does not emit error message to an LGPLed module.

Objections?


2012-01-10  Bruno Haible  <address@hidden>

        copy-file: Use 'quote' module consistently.
        * lib/copy-file.c (copy_file_preserving): Use quote().

        copy-file: Refactor.
        * lib/copy-file.c: Include quote.h.
        (copy_file_preserving): Call qcopy_acl instead of copy_acl. Emit error
        message here.
        * modules/copy-file (Depends-on): Add quote.

        acl: Export qcopy_acl.
        * lib/acl.h (qcopy_acl): New declaration.
        * lib/copy-acl.c (qcopy_acl): Make non-static.

        acl: Tweak.
        * lib/set-mode-acl.c (set_acl): Use same variable name as in copy_acl.

        acl: Align return values of copy_acl and qcopy_acl.
        * lib/copy-acl.c (copy_acl): Return the same value as qcopy_acl,
        maybe < -1.

0001-acl-Align-return-values-of-copy_acl-and-qcopy_acl.patch
Description: Text Data

0002-acl-Tweak.patch
Description: Text Data

0003-acl-Export-qcopy_acl.patch
Description: Text Data

0004-copy-file-Refactor.patch
Description: Text Data

0005-copy-file-Use-quote-module-consistently.patch
Description: Text Data