Re: quoting of strings in errors

[Pádraig Brady]

On 03/11/15 13:44, Pádraig Brady wrote:
> Done within this set of 8 related patches, available at:
> http://www.pixelbeat.org/patches/coreutils/coreutils-filename-escaping.patch

The patch set at the above URL is now 11 patches, with these added:

commit 0c9804f68fa540d9d9a2460860dff8a3c74a4a47
Author: Pádraig Brady <address@hidden>
Date:   Tue Nov 3 16:32:08 2015 +0000

    ls: default to --quoting=shell-escape for output to terminal

    * src/ls.c (decode_switches): Set "shell-escape" if isatty().
    * doc/coreutils.texi (ls invocation): Update the defaults description.
    * NEWS: Mention the change in behavior.  It should not have
    backwards compat issues, but mentioning here just in case.

commit 213396e9d66c73fb8de12089a673c80a68338d08
Author: Pádraig Brady <address@hidden>
Date:   Wed Nov 4 01:14:33 2015 +0000

    paste: avoid confusing backslash quoting in diagnostic

    * src/paste.c (main): Setting the quoting style to "escape"
    went against the intent of the comment about presenting
    doubled backslashes to the user.  Instead use "c-maybe"
    which is the only mode which avoids doubled backslashes,
    and provides protection against arbitrary control characters.
    * tests/misc/paste.pl: Adjust accordingly.

commit bc34105ef2ca412c18bcc20b2ee97e661bb4c895
Author: Pádraig Brady <address@hidden>
Date:   Wed Nov 4 02:44:41 2015 +0000

    maint: add a syntax check to avoid unstyled quoting

    * src/paste.c (main): Use our styled wrapper for quotearg_colon().
    * cfg.mk (sc_prohibit-quotearg): A new syntax check to avoid
    future uses of unstyled quotearg to one of the internal slots,
    and thus destined for diagnostic output.