From: Eric Sunshine <sunshine@sunshineco.com>
Date: Sun, 17 Mar 2013 08:59:49 GMT
Subject: Re: [PATCH 09/12] pretty: add %C(auto) for auto-coloring on the next placeholder
Message-ID: <CAPig+cTsq82zaEn8779vc3StD_-Ms2jgfGc4nhiB+EBTHA=+2w@mail.gmail.com>
In-Reply-To: <1363400683-14813-10-git-send-email-pclouds@gmail.com>

On Fri, Mar 15, 2013 at 10:24 PM, Nguyễn Thái Ngọc Duy
<pclouds@gmail.com> wrote:
> This is not simply convenient over $C(auto,xxx). Some placeholders

s/\$/%/

> (actually only one, %d) do multi coloring and we can't emit a multiple
> colors with %C(auto,xxx).
>
> diff --git a/Documentation/pretty-formats.txt b/Documentation/pretty-formats.txt
> index 66345d1..8734224 100644
> --- a/Documentation/pretty-formats.txt
> +++ b/Documentation/pretty-formats.txt
> @@ -154,7 +154,8 @@ The placeholders are:
>    adding `auto,` at the beginning will emit color only when colors are
>    enabled for log output (by `color.diff`, `color.ui`, or `--color`, and
>    respecting the `auto` settings of the former if we are going to a
> -  terminal)
> +  terminal). `auto` alone (i.e. `%C(auto)`) will turn on auto coloring

s/auto coloring/auto-coloring/

> +  on the following placeholder.