On Sun, Apr 25, 2010 at 17:42, Will Palmer <wmpalmer@xxxxxxxxx> wrote: > diff --git a/Documentation/pretty-formats.txt b/Documentation/pretty-formats.txt > index 1686a54..53eb903 100644 > --- a/Documentation/pretty-formats.txt > +++ b/Documentation/pretty-formats.txt > @@ -132,6 +132,7 @@ The placeholders are: > - '%Cblue': switch color to blue > - '%Creset': reset color > - '%C(...)': color specification, as described in color.branch.* config option > +- '%C?...: switch to specified color, if relevant color.* config option specifies that color is ok You missed the closing quote character. Besides, how do you think to distinguish between "%C?diff.color" and "%C?diff.colorcontinuation text"? The "%C(...)" has a placeholde termination rule (the closing bracket), yours does not seem to have one (unless something is missing in the documentation). Ah... I see. Definitely something missing. The spec should be: '%C?(...)'. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html