Re: [PATCH] Document and test the new % shotcut for the tracked branch

From: Johannes Schindelin <johannes.schindelin@gmx.de>
Date: Mar 20, 2009, 11:48 UTC
Message-ID: <alpine.DEB.1.00.0903201247440.6865@intel-tinevez-2-302>
In-Reply-To: <20090320111656.GA8940@machine.or.cz>

Hi,

On Fri, 20 Mar 2009, Petr Baudis wrote:

Show 13 quoted lines

> On Fri, Mar 20, 2009 at 11:31:31AM +0100, Johannes Schindelin wrote:
> > I had it ready yesterday!  But the real problem is not addressed by 
> > your patch, either: '%<branch>' is a legal branch name.
> > 
> > I briefly considered <branch>^{tracked}, but
> > 
> > - the ^{} codepath does not try to substitute branch _names_, so we'd 
> >   have to duplicate that ^{} detection, and,
> > 
> > - it is really cumbersome to write.
> 
> We already have @{N} to do one branch-based substitution, so what about 
> following this convention? Can't we have @{t}?

Have you ever tried a German keyboard layout? You would not say that @{} is easy to write if you did.

Besides, @{<string>} is already taken for the date-based reflog.

Ciao, Dscho