On Sun, Apr 21, 2013 at 2:02 PM, Andreas Schwab <schwab@xxxxxxxxxxxxxx> wrote: > Junio C Hamano <gitster@xxxxxxxxx> writes: > >> Andreas Schwab <schwab@xxxxxxxxxxxxxx> writes: >> >>> Felipe Contreras <felipe.contreras@xxxxxxxxx> writes: >>> >>>> + if [ -n "${ZSH_VERSION-}" ]; then >>>> + u="%%" >>>> + else >>>> + u="%" >>>> + fi >>> >>> aka u="%${ZSH_VERSION:+%}" >> >> Do you need/want a colon there? Even if it is empty, as long as it >> is defined we would want this, no? > > The original wouldn't. I thought you were trying to improve the original ;) -- Felipe Contreras -- 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