On Wed, Oct 7, 2015 at 11:28 PM, Matthieu Moy <Matthieu.Moy@xxxxxxxxxxxxxxx> wrote: > Karthik Nayak <karthik.188@xxxxxxxxx> writes: > >> On Wed, Oct 7, 2015 at 12:33 AM, Matthieu Moy >> <Matthieu.Moy@xxxxxxxxxxxxxxx> 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. I'll send version2 of the patch series to the list then :) -- Regards, Karthik Nayak -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html