Thread (64 messages) flat view 64 messages, 5 authors, 2016-06-15

Re: [PATCH 09/12] pretty: add %C(auto) for auto-coloring on the next placeholder

From: Eric Sunshine <hidden>
Date: 2016-06-15 22:56:25

On Fri, Mar 15, 2013 at 10:24 PM, Nguyễn Thái Ngọc Duy
[off-list ref] wrote:
This is not simply convenient over $C(auto,xxx). Some placeholders
s/\$/%/
quoted hunk ↗ jump to hunk
(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.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help