Re: [PATCH 8/9] branch: use ref-filter printing APIs
- From
Matthieu Moy <matthieu.moy@grenoble-inp.fr>
- Date
- Oct 7, 2015, 17:58 UTC
- Message-ID
- <vpqmvvu4l3o.fsf@grenoble-inp.fr>
- In-Reply-To
- <CAOLa=ZS4xVfsQ+F+KGD8Gz9NHzvJtuVvW16RSLV08ZiEv8DhKw@mail.gmail.com>
Karthik Nayak <karthik.188@gmail.com> writes:
Show 11 quoted lines
> On Wed, Oct 7, 2015 at 12:33 AM, Matthieu Moy > <Matthieu.Moy@grenoble-inp.fr> wrote: > >> To go a bit further, you can pre-build a string or strbuf aligned_short >> with value like "%%(align:20,left)%%(refname:short)%%(end)" and use it >> where needed (it's not a constant because you have to introduce maxwidth >> into the string, so it's not a candidate for #define). >> > > Again, the remote has a remote prefix here, so we can't really factor > it out much.
Ah, OK, there's a %s containing the remote prefix in "remote" that is not there in "local", and in the last version you sent it's already factored for verbose > 1 and verbose == 1. Indeed, there's nothing more to gain here.
-- Matthieu Moy http://www-verimag.imag.fr/~moy/