Re: [PATCH for "next"] pretty-formats: add 'format:<string>'

From: Johannes Schindelin <johannes.schindelin@gmx.de>
Date: Feb 24, 2007, 01:25 UTC
Message-ID: <Pine.LNX.4.63.0702240222100.22628@wbgn013.biozentrum.uni-wuerzburg.de>
In-Reply-To: <200702232053.49489.robin.rosenberg.lists@dewire.com>

Hi,

On Fri, 23 Feb 2007, Robin Rosenberg wrote:

Show 21 quoted lines

> fredag 23 februari 2007 02:07 skrev Johannes Schindelin:
> > 
> > On Fri, 23 Feb 2007, Han-Wen Nienhuys wrote:
> > 
> > > nitpick:
> > > 
> > >   \n
> > > 
> > > for newline would be nice. Similar for backslash, formfeed, alarm, 
> > > etc.
> > 
> > Yes, I thought about that. But it would change behaviour (even if I 
> > don't think it would do serious damage; the only user of 
> > interpolate.[ch] I saw is git-daemon, and that does not need \n, I 
> > guess).
>
> Other tools that come to mind, rpm and clearcase use \n vfor newline in 
> the format argument, which is good because I can guess that even without 
> looking at the documentation. %n I'd guess would be for a number of some 
> kind, e..g. the ordinal number of the commit listed (in subset and order 
> of the listed commits)

Okay. Patch?

Show 5 quoted lines

> > Besides, "%n" is
> > 
> > - more consistent,
>
> with...?

... itself? Why should not _one_ escape character be enough?

> > - you can put BS, FF, AL, etc. into the format string before passing
> >   it as an option to git; git does not have to help you there.
>
> They are hard to type in shells and even harder in gui's.

You would not do that all that often, but rather write a script. Even the config format allows for inclusion of special characters, so aliases should be fine.

Ciao, Dscho