Jan Krüger <jk@xxxxx> writes: > On 02/27/2012 11:28 PM, Junio C Hamano wrote: >> I think this one (unlike 1/2) makes sense, but a single letter -s feels a >> bit too vague. Always spelling in long option "--short" so that it >> matches "%(refname:short)" in for-each-ref might be better, I would think. >> >> Especially given that the expected use case is primarily in scripts not >> from the command line, being more explicit and easier to read has value >> over being short and easier to (mis)type. > > Good point. If we can agree on what to do with the first patch, I'll > change that in the next iteration. Dropping the first patch would be my preference. I do not think this change deserves to be taken hostage to it. -- 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