Re: Make "terminator behavior" the default with --pretty=format: ?

On Tue, Feb 22, 2011 at 11:51 AM, Hrvoje Nikšić [off-list ref] wrote:

On Tue, Feb 22, 2011 at 5:43 PM, Will Palmer [off-list ref] wrote:

quoted

On Tue, 2011-02-22 at 16:43 +0100, Hrvoje Nikšić wrote:

quoted

Is there a reason, other than backward compatibility, for
"--prety=format:" to have separator rather than terminator semantics?

The "default behaviour" is the behaviour which occurs when one /doesn't/
specify something. For example: --pretty="%H %an" uses terminator
semantics.

I didn't know that you could simply omit the "format:". Is it
documented anywhere? The docs say:

      --pretty[=<format>], --format[=<format>]
          Pretty-print the contents of the commit logs in a given
format, where <format> can be one of oneline, short,
          medium, full, fuller, email, raw and format:<string>. When
omitted, the format defaults to medium.

In the tformat section is:

 "In addition, any unrecognized string that has a % in it is
interpreted as if it has tformat: in front of it."

Of course, you need to know to read that far into the man page to find
that sentence, and the earlier summary of --pretty never gives any
hint to do so.

Mind submitting a documentation patch?

j.