Re: [PATCH] Extend "checkout --track" DWIM to support more cases

From: Alex Riesen <raa.lkml@gmail.com>
Date: Aug 20, 2008, 20:04 UTC
Message-ID: <20080820200440.GF16626@blimp.local>
In-Reply-To: <alpine.DEB.1.00.0808202151320.24820@pacific.mpi-cbg.de.mpi-cbg.de>

Johannes Schindelin, Wed, Aug 20, 2008 21:52:23 +0200:

Show 10 quoted lines

> On Wed, 20 Aug 2008, Alex Riesen wrote:
> > -		slash = strchr(argv[0], '/');
> > -		if (slash && !prefixcmp(argv[0], "refs/"))
> > -			slash = strchr(slash + 1, '/');
> > -		if (slash && !prefixcmp(argv[0], "remotes/"))
> > -			slash = strchr(slash + 1, '/');
> 
> Why is this not enough?  It strips refs/ if there is one, and remotes/ if 
> there is one (possibly after stripping refs/).  No?
>

No. It strips refs/ OR remotes/ (because of prefixcmp with argv[0]). And I still wanted refs/<namespace>/something...